Ticket #3505 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Change to workspace index for vertical axis of color fill plots

Reported by: Vickie Lynch Owned by: Martyn Gigg
Priority: major Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

Some SNS instrument have large gaps in the detectorIDs that cause the color fill plot to have large gaps now that the spectra numbers have changed. See attached picture of POWGEN calibration. Plots would look better with workspace index used for vertical axis.

Attachments

PG3_2538.png​ (233.9 KB) - added by Vickie Lynch 9 years ago.
Compare to plot in wiki on CalibrateRectangularDetectors page

Change History

Changed 9 years ago by Vickie Lynch

Compare to plot in wiki on CalibrateRectangularDetectors page

comment:1 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted
  • Owner set to Martyn Gigg

comment:2 Changed 9 years ago by Martyn Gigg

As discussed with Russell we are going to revert back to spectrum numbers starting at 1 and handle the ISIS specific stuff in the loaders. This should give you back a contiguous plot.

comment:3 Changed 9 years ago by Martyn Gigg

In [13881]:

Refs #3505. Revert to always starting spectrum numbers at 1 and handle ISIS specific spectrum stuff in the event loaders.

comment:4 Changed 9 years ago by Vickie Lynch

Color fill plots from SNS calibrations look great again. Thanks!

comment:5 Changed 9 years ago by Martyn Gigg

In [13884]:

Refs #3505 Fix a Python test.

comment:6 Changed 9 years ago by Martyn Gigg

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

comment:7 Changed 9 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:8 Changed 9 years ago by Russell Taylor

  • Status changed from verifying to closed

Colorfill plot after a LoadEventNexus on an instrument with non-contiguous detector ID's look lovely and contiguous.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4352

Note: See TracTickets for help on using tickets.