Ticket #1600 (closed: fixed)
Usability Changes in Indirect ConvertToEnergy
Reported by: | Michael Whitty | Owned by: | Michael Whitty |
---|---|---|---|
Priority: | major | Milestone: | Iteration 25 |
Component: | MantidPlot | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Anders Markvardsen |
Description
Based on feedback from Mr Howells - various adjustments
Change History
comment:2 Changed 10 years ago by Michael Whitty
(In [6082]) re #1600 - "Plot Raw" button now behaves more consistently with invalid input; Calibration file input is now switched between being "optional" depending on whether the "Use Calib" checkbox is checked so the ui won't flag it as invalid when the user doesn't wish to use one; RES output will now be plotted if the "Plot Result" is checked, otherwise the workspace will be cleared away; the "Run" button is now disabled on the Calibration tab to avoid possible confusion
comment:3 Changed 10 years ago by Michael Whitty
(In [6097]) re #1600 - adjusting the process of switching between direct/indirect to take into account both interfaces connect their own slot to the "Browse" button for saving location, so if a user had used (initialised) both interfaces, they would get two separate pop-ups asking for different things. the switching process now disconnects/reconnects the appropriate signals.
comment:6 Changed 10 years ago by Michael Whitty
(In [6261]) re #1600 and #1611 - various adjustments to the indirect cte/indirect analysis uis. adding parameter files for diffraction spectra with aim to developing support for these shortly (as part of indirectAnalysis initially), adjustment of layout so each tab on IDA has three checkboxes (Verbose, Plot, Save) consistently, though Verbose isn't done yet. Added plot functions to slice, elwin and absorption. adjustments to how validators appear on indirect CTE when they're not actually required. disabling tabs that don't fit the current analyser/reflection selection.
also adding some files to the 2010 VS project that were lacking and preventing building locally
comment:11 Changed 10 years ago by Michael Whitty
comment:12 Changed 10 years ago by Michael Whitty
comment:13 Changed 10 years ago by Michael Whitty
comment:14 Changed 10 years ago by Michael Whitty
comment:15 Changed 10 years ago by Michael Whitty
comment:16 Changed 10 years ago by Michael Whitty
comment:17 Changed 10 years ago by Michael Whitty
comment:18 Changed 10 years ago by Michael Whitty
comment:19 Changed 10 years ago by Michael Whitty
comment:20 Changed 10 years ago by Michael Whitty
comment:21 Changed 10 years ago by Michael Whitty
comment:22 Changed 10 years ago by Michael Whitty
comment:23 Changed 10 years ago by Michael Whitty
- Status changed from accepted to verify
- Resolution set to fixed
mark complete for this ticket, more will surely follow in next iteration
comment:24 Changed 10 years ago by Anders Markvardsen
- Status changed from verify to verifying
- Tester set to Anders Markvardsen
comment:25 Changed 10 years ago by Anders Markvardsen
- Status changed from verifying to closed
Changes made to interfaces looks good including plotting of S(Q,omega) (next release will see spectrum number being swapped with Q).
As a suggestion for future improvement when in 'Energy Transfer' tab e.g. instrument OSIRIS is selected and in the 'Calibration' tab a IRIS file is selected using the Browse button, then when plotted using the 'Plot Raw' button currently the Script Console is opened with an error message. MantidPlot does not crash but consider presenting the user with a more soft message.
comment:26 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 2447