mantid icon indicating copy to clipboard operation
mantid copied to clipboard

Superplot: Normalise by Max Issues

Open cailafinn opened this issue 1 year ago • 2 comments

Describe the bug Some issues with the "Normalise by max" feature of "Superplot":

  • [ ] It is a little ambiguous from the name or the documentation that this will normalise every curve individually

    From the docs:

    By using the dedicated checkbox above this list, one can toggle the activation of the normalisation (by the maximum value) of the curves.

    This could also be interpreted as meaning that all curves will be scaled by the same amount (i.e. from the maximum value across all curves)

  • [ ] If the superplot button is clicked twice to close and re-open that section, the normalisation remains active but the box is no longer ticked

  • [ ] https://github.com/mantidproject/mantid/issues/36103

  • [ ] https://github.com/mantidproject/mantid/issues/36104

To Reproduce

  1. Plot multiple spectra
  2. Click the superplot button
  3. Enable normalisation
  4. Click the superplot button to close it
  5. Reopen the superplot menu

Expected behavior

  • The documentation or tooltips should be made more clear
  • The state of the checkbox should be maintained between closing and opening the superplot

Screenshots image

Platform/Version (please complete the following information):

  • OS: All

cailafinn avatar May 18 '23 10:05 cailafinn

I've added some issues from 6.8 manual testing

thomashampson avatar Sep 14 '23 23:09 thomashampson

Has this been completed or does the milestone need changing?

sf1919 avatar Jan 22 '24 14:01 sf1919