Ticket #4006 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

ManitdPlot: Move the detector table to use getSpectrum over the spectraAxis

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

For consistency with most of the code now and assuming that the spectra axis will go away at some point?

To the tester: Test this by making sure that the spectrum numbers are correct when viewing the detector table. It is possible best to do this with an ISIS file as it may be more obvious that something is wrong. It would be a good idea to test the raw loaded file and then after some algorithms check that the spectrum numbers remain intact.

Change History

comment:1 Changed 9 years ago by Martyn Gigg

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

Fixes #4006. Update the detector table code to use getSpectrum. That way it is consistent with more of the algorithms and the getDetector call later in the code.

Changeset: 14b82d450060b31ecea740e9515372fd6860ef44

comment:2 Changed 9 years ago by Martyn Gigg

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:3 Changed 9 years ago by Martyn Gigg

  • Status changed from reopened to accepted

comment:4 Changed 9 years ago by Martyn Gigg

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

comment:5 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:6 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

Using GEM38370 and converting to different units a few time, it looks like spectrum numbers are intact. At least the range defaults in plot spectrum and show data seem to be consistent.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4853

Note: See TracTickets for help on using tickets.