Ticket #2974 (closed: fixed)
In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us) and a few other changes.
Reported by: | Anders Markvardsen | Owned by: | Robert Whitley |
---|---|---|---|
Priority: | major | Milestone: | Iteration 30 |
Component: | MantidPlot | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Martyn Gigg |
Description
See attached picture for proposed layout of fitting tab.
Note before this ticket can be completed ticket #2972 needs to have been completed.
Attachments
Change History
comment:1 Changed 9 years ago by Anders Markvardsen
Comment from Steve about Fit raw data or bunch data:
If no bunching (binning) has been defined elsewhere (in plot options?) then this menu probably wouldn’t have the option to fit ‘bunch data’. Other possibilities are to place a ‘Fit bunched data’ tick box next to the bunching instruction in plot options, in this case, you’d still need a line in the analysis tab to report what the program is doing, or to define a separate bunching at this point for data fitting. I guess I prefer the first of the ‘other options’, but not too strongly. We seem to have called ‘bunching’ ‘binning’ in the plot options, so we’ll need to be consistent.
comment:2 Changed 9 years ago by Anders Markvardsen
More comments from Steve:
- We’ve made a deal about hiding Workspaces while using the muon interface. On the other hand, it might be good to know which Workspace the program is using. Perhaps the Workspaces should be reported as read only, the minimizer definable.
- Units are needed on the startx and endx values (the use of tooltips for the fit parameters sounds a very good idea).
comment:3 Changed 9 years ago by Anders Markvardsen
Most recent comment from Muon guys on startX and endX
We had a bit of a debate about using ‘time’ or ‘x’ (‘x’ could be ‘time’ or ‘freq’ was the discussion). However, we seem to have specialised in other parts of the interface on analysing the time domain, so perhaps keep it identical with the ‘plot options’ … ‘Start at (us)’, etc
comment:4 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 29 to Iteration 30
"New" tickets moved at the code freeze of iteration 29
comment:5 Changed 9 years ago by Anders Markvardsen
- Status changed from new to assigned
- Owner changed from Anders Markvardsen to Robert Whitley
Some the stuff in this ticket has been fixed, but we have a chat about the rest when u get back
comment:6 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 30 to Iteration 31
Bulk move of tickets to iteration 31 at the iteration 30 code freeze
comment:7 Changed 9 years ago by Anders Markvardsen
- Summary changed from MuonAnalysis Fitting tab to In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us)
comment:8 Changed 9 years ago by Robert Whitley
Talking to the scientists they have asked for a different layout for the fit property browser within the data analysis tab. The layout is as such.
Functions
(as is)
Data
Workspace
Workspace index
Start (us)
Finish (us)
Settings
Minimizer
Plot Difference
Fit to binned data? (yes/no, default ‘no’ or perhaps a check box)
comment:10 Changed 9 years ago by Robert Whitley
- Status changed from assigned to accepted
- Summary changed from In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us) to In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us) and a few other changes to the FitPropertyBrowser layout
comment:11 Changed 9 years ago by Robert Whitley
- Summary changed from In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us) and a few other changes to the FitPropertyBrowser layout to In MuonAnalysis Fitting tab change startX to Start (us) and endX to Finish (us) and a few other changes.
comment:12 Changed 9 years ago by Robert Whitley
- Status changed from accepted to verify
- Resolution set to fixed
comment:14 Changed 9 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:15 Changed 9 years ago by Martyn Gigg
- Status changed from verifying to closed
Confirmed that the fields have different names on the Muon tab. However, I'm not convinced what has been implemented is the best long-term solution as it relies on having custom code for the Muon (and possibly others in the future) group in the general interface.
A better solution may be to specialise the FitPropertyBrowser and set the appropriate differences in this class.
comment:16 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3821