Ticket #5487 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Incorporate Calibration information into the SCD workflow

Reported by: Ruth Mikkelson Owned by: Ruth Mikkelson
Priority: major Milestone: Release 2.3
Component: Mantid Keywords: SCD,Calibration
Cc: Blocked By:
Blocking: Tester: Dennis Mikkelson

Description (last modified by Dennis Mikkelson) (diff)

The LoadIsawPeaks algorithm must incorporate the calibrated instrument information that's written at the top of the peaks file. The detector position information from the peaks file must be used to update the positions of the detector pixels so the corrected positions are also used by other algorithms.

Change History

comment:1 Changed 8 years ago by Ruth Mikkelson

  • Status changed from new to accepted
  • Owner set to Ruth Mikkelson

comment:2 Changed 8 years ago by Ruth Mikkelson

Refs #5487 LoadIsawPeaks now incorporates Calibration info

Values in test programs were slighly off. Also, changes were needed when the instrument was parameterized from the start.

Changeset: 23894feec97044c7960fe22fe19c4b6284f9be2a

comment:3 Changed 8 years ago by Ruth Mikkelson

Refs #5487 LoadIsawPeaks now incorporates Calibration info

Values in test programs were slighly off. Also, changes were needed when the instrument was parameterized from the start.

Changeset: 23894feec97044c7960fe22fe19c4b6284f9be2a

comment:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:5 Changed 8 years ago by Ruth Mikkelson

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

comment:6 Changed 8 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:7 Changed 8 years ago by Vickie Lynch

  • Status changed from verifying to verify
  • Tester Vickie Lynch deleted

comment:8 Changed 8 years ago by Russell Taylor

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

comment:9 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to verify
  • Component changed from Infrastructure to Mantid
  • Tester Russell Taylor deleted

comment:10 Changed 8 years ago by Dennis Mikkelson

  • Status changed from verify to verifying
  • Tester set to Dennis Mikkelson

comment:11 Changed 8 years ago by Dennis Mikkelson

  • Status changed from verifying to closed
  • Description modified (diff)

This ticket was tested as follows. TOPAZ 5637 was loaded and converted to a diffraction MD workspace. Find peaks MD was used to find 500 peaks. FindUBUsingFFT then reported that 490 of 500 peaks would be indexed to within 0.1. To test the use of calibration information, an ISAW DetCal file with more accurate detector positions was loaded into the TOPAZ 5637 event file. After converting this calibrated workspace to a diffraction MD workspace, peaks were again found and indexed. In the peaks workspace using calibrated data, 496 of 500 peaks were indexed to within 0.1, so the calibration gave more accurate peak positions. Finally, the calibrated peaks workspace was saved using SaveIsawPeaks and loaded to a different workspace using LoadIsawPeaks. When FindUBUsingFFT was used on the reloaded peaks workspace, 496 of 500 peaks were again indexed to within 0.1, indicating that the calibration information from the top of the peaks file was used by LoadIsawPeaks, rather than the default detector positions.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6333

Note: See TracTickets for help on using tickets.