Ticket #1755 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Powder diffraction calibration for SNAP

Reported by: Peter Peterson Owned by: Vickie Lynch
Priority: major Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Janik Zikovsky

Description

Since SNAP moves its detectors (and sample) frequently they are not able to get good enough statistics to run the cross-correlation algorithm. Create a new algorithm that maximizes the total counts in a specific d-spacing window by modifying the position of the, fixed size, area detectors.

Attachments

DetCal_SNAP_4307.png​ (311.6 KB) - added by Vickie Lynch 10 years ago.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Owner set to Peter Peterson
  • Status changed from new to assigned

comment:2 Changed 10 years ago by Peter Peterson

  • Milestone changed from Iteration 25 to Iteration 26

comment:3 Changed 10 years ago by Vickie Lynch

  • Status changed from assigned to accepted
  • Owner changed from Peter Peterson to Vickie Lynch

comment:4 Changed 10 years ago by Vickie Lynch

(In [7768]) Refs #1755 Optimization routine for calibrating SNAP detectors

comment:5 Changed 10 years ago by Vickie Lynch

(In [7786]) Refs #1755 Needed another workspace for calibration

comment:6 Changed 10 years ago by Vickie Lynch

(In [7787]) Refs #1755 Deleted unnecessary lines in GSLFunctions.h

comment:7 Changed 10 years ago by Vickie Lynch

(In [7809]) Refs #1755 Using simplex instead of simplex2 for earlier gsl version on RHEL5

comment:8 Changed 10 years ago by Vickie Lynch

(In [7824]) Refs #1755 Corrected author for DiffractionEventCalibrateDetectors

comment:9 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:10 Changed 10 years ago by Vickie Lynch

(In [8729]) Refs #1755 Removed log information for algorithms called by DiffractionEventCalibrateDetectors; added parallel loop for all detectors

comment:11 Changed 10 years ago by Vickie Lynch

(In [8748]) Refs #1755 new function to avoid repeating calls to move detectors in DiffractionEventCalibrateDetectors

comment:12 Changed 10 years ago by Vickie Lynch

(In [8752]) Refs #1755 Fixed spelling mistake; deleteted unused lines

comment:13 Changed 10 years ago by Vickie Lynch

(In [8787]) Refs #1755 Make bin boundaries for rebinning an input option for DiffractionEventCalibrateDetectors

comment:14 Changed 10 years ago by Vickie Lynch

(In [8825]) Refs #1755 Add Sort and replace AlignDetectors with ConvertUnits in DiffractionEventCalibrateDetectors

comment:15 Changed 10 years ago by Vickie Lynch

(In [9147]) Refs #1755 Changed argument for CreateCalFileByName in DiffractionEventCalibrateDetectors to match Janik's changes in algorithm

comment:16 Changed 10 years ago by Vickie Lynch

(In [9148]) Refs #1755 Changed Value to MatrixWorkspace for setProperty

comment:17 Changed 10 years ago by Vickie Lynch

(In [9250]) Refs #1755 Fix compiler warnings; don't scale positions; relative rotations for DiffractionEventCalibrateDetectors

comment:18 Changed 10 years ago by Vickie Lynch

(In [9324]) Refs #1755 Optimize on both height and position of peak

comment:19 Changed 10 years ago by Vickie Lynch

(In [9394]) Refs #1755 Weighted peak height in optimization cost function for good results in SNAP_4307

comment:20 Changed 10 years ago by Vickie Lynch

(In [9430]) Refs #1755 Write DetCal file after calibration

comment:21 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:22 Changed 10 years ago by Vickie Lynch

(In [9562]) Refs #1755 DetCal file is correctly written now

comment:23 Changed 10 years ago by Vickie Lynch

(In [9575]) Refs #1755 Fit peak with Gaussian for calibration

comment:24 Changed 10 years ago by Vickie Lynch

(In [9577]) Refs #1755 Fixed arguments for StartX adn EndX

comment:25 Changed 10 years ago by Vickie Lynch

(In [9606]) Refs #1755 Option for only calibrating one detector so we can use slurm

comment:26 Changed 10 years ago by Vickie Lynch

(In [9670]) Refs #1755 Scale cost function by number of events; Correct data in DetCal file

Changed 10 years ago by Vickie Lynch

comment:27 Changed 10 years ago by Vickie Lynch

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

Runs in approximately 20 minutes per detector with events for SNAP.

comment:28 Changed 9 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:29 Changed 9 years ago by Janik Zikovsky

  • Status changed from verifying to closed

Tested working on SNAP data. Took 5 hours; will try to re-enable the parallel loop since a change I made might prevent the segfault...

comment:30 Changed 9 years ago by Janik Zikovsky

Nevermind, memory use killed me! :)

comment:31 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2602

Note: See TracTickets for help on using tickets.