Ticket #3705 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SNSSingleCrystalReduction corrections

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Shelly Ren

Description

Normalize all data to beam monitor count before subtracting or dividing. Subtract background from vanadium. Integrate vanadium in TOF and also divide by integral. Option to smooth vanadium with SmoothNeighbours. Do not delete background and vanadium workspace to save time with multiple samples.

Attachments

SNSSingleCrystalReduction.png (204.0 KB) - added by Vickie Lynch 9 years ago.

Change History

comment:1 Changed 9 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Vickie Lynch

In [14588]:

Refs #3705 Correction changes suggested by TOPAZ team

comment:3 Changed 9 years ago by Vickie Lynch

In [14595]:

Refs #3705 Sort events before subtraction

comment:4 Changed 9 years ago by Vickie Lynch

In [14740]:

Refs #3705 Option to zero edges of rectangular detectors when smoothing

comment:5 Changed 9 years ago by Vickie Lynch

In [14741]:

Refs #3705 Correct one bank at a time; normalize by upstream monitor

comment:6 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:7 Changed 9 years ago by Vickie Lynch

In [14812]:

Refs #2614 and #3705 Masking problems for division by vanadium bank by bank fixed

comment:17 Changed 9 years ago by Vickie Lynch

In [14881]:

Refs #3705 Process vanadium first, apply anvred corrections, option to integrate peaks and save hkl file

comment:18 Changed 9 years ago by Vickie Lynch

In [14887]:

Refs #3705 Option to append hkl file added; do not do AnvredCorrections if Radius = 0

comment:19 Changed 9 years ago by Vickie Lynch

In [14888]:

Refs #3705 Correct variable name

comment:20 Changed 9 years ago by Vickie Lynch

In [14891]:

Refs #3705 Fix if test

comment:21 Changed 9 years ago by Vickie Lynch

In [14937]:

Refs #3705 Correct extension; scale data for fitting routine to work

comment:22 Changed 9 years ago by Vickie Lynch

In [14974]:

Refs #3705 Change name of UB matrix file for each sample; releaseFreeMemory after integrating peak

comment:23 Changed 9 years ago by Vickie Lynch

In [15008]:

Refs #3705 almost 10% decrease in time sorting after each ConvertUnits

comment:24 Changed 9 years ago by Vickie Lynch

In [15013]:

Refs #3705 Option to append hkl file added

comment:25 Changed 9 years ago by Vickie Lynch

In [15020]:

Refs #3705 Should not CompressEvents in d-space

comment:26 Changed 9 years ago by Vickie Lynch

In [15057]:

Refs #3705 Save time by converting units inside AnvredCorrection

comment:27 Changed 9 years ago by Vickie Lynch

In [15188]:

Refs #3705 Results match ISAW anvred results now

comment:28 Changed 9 years ago by Vickie Lynch

In [15194]:

Refs #3705 Add options for Wavelength range and minimum dSpacing

comment:29 Changed 9 years ago by Vickie Lynch

In [15201]:

Refs #3705 Options for more corrections to data

comment:30 Changed 9 years ago by Vickie Lynch

In [15213]:

Refs #3705 Input scale factor; normalize vanadium integrated by pixel

comment:31 Changed 9 years ago by Vickie Lynch

In [15222]:

Refs #3705 Use peaks from PredictPeaks; save integrated monitor count from sample when subtracting background

comment:32 Changed 9 years ago by Vickie Lynch

In [15256]:

Refs #3705 Only load one back of background

comment:33 Changed 9 years ago by Vickie Lynch

In [15352]:

Refs #3705 Need ub matrix not goniometer * ub matrix

comment:34 Changed 9 years ago by Vickie Lynch

In [15368]:

Refs #3705 Rhombohedrally centered reflection condition for peaks

comment:26 Changed 9 years ago by Vickie Lynch

Refs #3705 Input option for number of edge pixels

Changeset: 05c1f7100c5d1eafa6028b0844a2d226df1521e8

comment:27 Changed 9 years ago by Vickie Lynch

Refs #3705 Input option for number of edge pixels

Changeset: 05c1f7100c5d1eafa6028b0844a2d226df1521e8

comment:28 Changed 9 years ago by Vickie Lynch

Refs #3705 Save in SNSSingleCrystalReduction has hkl as the default value

Changeset: 514aad414742273ac88972f2e2599cf0f8616902

comment:29 Changed 9 years ago by Vickie Lynch

Refs #3705 Input lattice parameters to calculate UB matrix

Changeset: 45cfe0a1148074137b8c85ff53ca4cf9491e1f62

comment:30 Changed 9 years ago by Vickie Lynch

Refs #3705 Lattice parameters all same type

Changeset: 7cc84fb6bc13d97a8fdb0efa7591fade1e6594a4

comment:31 Changed 9 years ago by Vickie Lynch

Refs #3705 Find, index, and integrate peaks for each bank separately

Changeset: 40ac1b031108929d9baf34a662441efadd7ce581

comment:32 Changed 9 years ago by Vickie Lynch

Refs #3705 Only load pixels for one bank; remove nxs output option

Changeset: c3b9a5fbecfe62a41868b941637432f4f78c5003

comment:33 Changed 9 years ago by Vickie Lynch

Refs #3705 Changes to index peaks better

Changeset: b0f9b86a64f1d7c99830e6aed7bb86a1279e7dad

comment:34 Changed 9 years ago by Vickie Lynch

Refs #3705 Start with same UB matrix for indexing all orientations

Changeset: 85bb5c529192afb7523bf052d900b812080b196b

comment:35 Changed 9 years ago by Vickie Lynch

Refs #3705 Changes to index peaks better

Changeset: b0f9b86a64f1d7c99830e6aed7bb86a1279e7dad

comment:36 Changed 9 years ago by Vickie Lynch

Refs #3705 Start with same UB matrix for indexing all orientations

Changeset: 85bb5c529192afb7523bf052d900b812080b196b

Changed 9 years ago by Vickie Lynch

comment:37 Changed 9 years ago by Vickie Lynch

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

See attached screenshot for testing.

Peaks in hkl file should be similar to ISAW's output: /SNS/TOPAZ/shared/Sapphire/NewFlatField/8Sept2011/intSapphire.hkl

comment:38 Changed 9 years ago by Shelly Ren

  • Status changed from verify to verifying
  • Tester set to Shelly Ren

comment:39 Changed 9 years ago by Vickie Lynch

Refs #3705 Fix file not loading using findnexus

Changeset: c9a03cc4d8bbb59468cf8f2653944909d7946263

comment:40 Changed 9 years ago by Vickie Lynch

Refs #3705 Extra lines not needed

Changeset: 405810447ca337befb762c1a202db53fac91d796

comment:41 Changed 9 years ago by Shelly Ren

  • Status changed from verifying to closed

Used the same input data shown in the attachment for testing, compared some results in the output .hkl file with the isaw output and they look fairly close.

comment:42 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4552

Note: See TracTickets for help on using tickets.