Ticket #9564 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Reverse the 2theta-order of POLDI spectra on loading

Reported by: Michael Wedel Owned by: Michael Wedel
Priority: major Milestone: Release 3.2
Component: Diffraction Keywords: POLDI
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

POLDI spectra are stored in descending 2theta-order, while all data analysis assumes ascending order (correlation method, 2D-spectrum calculation, possibly more). At this point it is feasible to change the loading behaviour, because only PoldiAutoCorrelationCore needs to be changed and it will be possible to use it for analysing simulated spectra as well.

Attachments

6904_before.png (93.2 KB) - added by Michael Wedel 6 years ago.
6904_after.png (91.8 KB) - added by Michael Wedel 6 years ago.

Change History

comment:1 Changed 6 years ago by Michael Wedel

Refs #9564. Reversed direction of POLDI data.

Unit tests had to be adjusted because of element ordering. While looking at the file, made some performance improvements.

Changeset: 4eb1ba077c3b3fbc8acdf6c67d4f03663363a254

Changed 6 years ago by Michael Wedel

Changed 6 years ago by Michael Wedel

comment:2 Changed 6 years ago by Michael Wedel

Testing information:

Find the file systemtests/Data/poldi2013n006904.hdf and make sure Mantid can locate it (or use the full path for the following example).

data_6904 = LoadSINQFile("poldi2013n006904.hdf", Instrument = "POLDI")

Create a Color fill plot of the workspace. Before the change it looks like in 6904_before.png, after the change like 6904_after.png. A good point for orientation is the one spectrum which is very bright due to a mulfunctioning wire.

Last edited 6 years ago by Michael Wedel (previous) (diff)

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:4 Changed 6 years ago by Michael Wedel

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

comment:5 Changed 6 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:6 Changed 6 years ago by Nick Draper

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9564_reverse_reading_order_poldi_data'

Full changeset: 4aa765d7cbf07c6219063986d43057c05f310373

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10407

Note: See TracTickets for help on using tickets.