Ticket #2974 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

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

Fitting-tab update.png (44.7 KB) - added by Anders Markvardsen 9 years ago.

Change History

Changed 9 years ago by Anders Markvardsen

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:

  1. 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.
  1. 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)

Last edited 9 years ago by Robert Whitley (previous) (diff)

comment:9 Changed 9 years ago by Robert Whitley

In [14842]:

Refs #2974. Changed the layout of the FitPropertyBrowser in the muon analysis interface and changed startX to Start (μs) and a similar thing for endX. I've also changed the data option to a more sensibly named "Fit to binned data?" one. This is a check box option and is defaulted to false.

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:13 Changed 9 years ago by Martyn Gigg

  • Milestone changed from Iteration 31 to Iteration 30

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

Note: See TracTickets for help on using tickets.