Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QT5 GUI - rename FitPage0 to Fitpage1 or rename M1 to M0 (Trac #1136) #1177

Closed
RichardHeenan opened this issue Mar 30, 2019 · 5 comments
Closed
Assignees
Labels
Defect Bug or undesirable behaviour Major Big change in the code or important change in behaviour
Milestone

Comments

@RichardHeenan
Copy link
Contributor

RichardHeenan commented Mar 30, 2019

The first !FitPage in the QT gui is FitPage0 but has models M1 and parameters M1.radius etc. This will make constraints very confusing!

Suggest rename one or the other, preferably to my mind start at FitPage1 for backwards compatibility with earlier versions.

(putting this in beta approx project for now, though is not really part of this)

Migrated from http://trac.sasview.org/ticket/1136

{
    "status": "closed",
    "changetime": "2018-08-21T11:15:08",
    "_ts": "2018-08-21 11:15:08.987585+00:00",
    "description": "The first !FitPage in the QT gui is FitPage0 but has models M1 and parameters M1.radius etc. This will make constraints very confusing!\n\nSuggest rename one or the other, preferably to my mind start at FitPage1 for backwards compatibility with earlier versions.\n\n(putting this in beta approx project for now, though is not really part of this)",
    "reporter": "richardh",
    "cc": "",
    "resolution": "fixed",
    "workpackage": "Beta Approximation Project",
    "time": "2018-07-18T16:37:42",
    "component": "SasView",
    "summary": "QT5 GUI - rename FitPage0 to Fitpage1 or rename M1 to M0",
    "priority": "major",
    "keywords": "",
    "milestone": "SasView 5.0.0",
    "owner": "richardh",
    "type": "defect"
}
@RichardHeenan RichardHeenan added this to the SasView 5.0.0 milestone Mar 30, 2019
@RichardHeenan RichardHeenan self-assigned this Mar 30, 2019
@RichardHeenan RichardHeenan added Beta Approximation Project Defect Bug or undesirable behaviour Major Big change in the code or important change in behaviour labels Mar 30, 2019
@sasview-bot
Copy link

Trac update at 2018/07/19 08:27:43: tcbennun commented:

I agree with starting at FitPage1, and it's a one-line fix, so I think I'll be bold and put it through...

@sasview-bot
Copy link

Trac update at 2018/07/19 08:41:11: tcbennun commented:

done: 6b75aee

@rozyczko
Copy link
Member

Trac update at 2018/07/20 08:13:46:

  • piotr changed _comment0 from "FittingPerspectiveTest.py will also need to modified" to "1532074491883495"
  • piotr commented:

FittingPerspectiveTest.py will also need to modified

@sasview-bot
Copy link

Trac update at 2018/07/20 08:48:25: tcbennun commented:

Replying to [comment:3 piotr]:

FittingPerspectiveTest.py will also need to modified
Of course, cheers. Done.

@RichardHeenan
Copy link
Contributor Author

Trac update at 2018/08/21 11:15:08:

  • richardh commented:

Has been done, so closing this ticket.

  • richardh changed resolution from "" to "fixed"
  • richardh changed status from "new" to "closed"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Bug or undesirable behaviour Major Big change in the code or important change in behaviour
Projects
None yet
Development

No branches or pull requests

3 participants