Ticket #924 (closed: fixed)

Opened 11 years ago

Last modified 5 years ago

Instrument view: Problems with the min colour map range in log scale

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: major Milestone: Iteration 24
Component: Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

a couple of problems:

  1. If you increase the min toward the max it truncates the colour map rather than increasing the resolution of the plotted data. (does not happen in the linear scale)
  2. You can set the min to greater than the max (this may also be possible in the linear scale).

Change History

comment:1 Changed 10 years ago by Martyn Gigg

  • Status changed from new to verify
  • Resolution set to fixed

(In [5935]) Fix log scaling on the instrument widget. Also removed useless high-resolution lighting option. Fixes #1534 and #924

comment:2 Changed 10 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:3 Changed 10 years ago by Peter Peterson

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The log color works as advertised. Unfortunately part 2 (min>max) is still allowed in the gui.

comment:4 Changed 10 years ago by Martyn Gigg

  • Status changed from reopened to verify
  • Resolution set to fixed

(In [5942]) Fix colour map on the instrument window so that minimum values cannot be greater than maxmimum and maximum not smaller than minimum. Fixes #924

comment:5 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester changed from Peter Peterson to Michael Whitty

comment:6 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

in [5943] - the gui silently ignores any invalid input and reverts the changes to their last legal value

making calls through python also dissalows illegal values, but no message or exception is thrown to alert the user to their mistake - not sure if we'd want such a message?

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​1772

Note: See TracTickets for help on using tickets.