Ticket #11588 (assigned)

Opened 5 years ago

Last modified 5 years ago

Add Vanadium consideration into EnginX algorithms

Reported by: Lottie Greenwood Owned by: Federico M Pouzols
Priority: major Milestone: Release 3.5
Component: Diffraction Keywords:
Cc: saurabh.kabra@… Blocked By:
Blocking: #8368 Tester:

Description (last modified by Lottie Greenwood) (diff)

Currently EnginX CalibrateAll does not consider the vanadium number when calibrating. This is necessary to get useful calibration results.

In order to do this, there are two methods:

Method one

1) Add in option to load the ceria and vanadium numbers

2) Fit a spline to each pixel (V-Nb) and store them with vanadium in a filename

3) Divide each Ceria pixel by the spline

Method two

1) Add in option to load the ceria and vanadium numbers

2) Add together each V-Nb pixel to get 1 number (the sensitivity) and store it

3) Divide the ceria pixel with corrected sensitivity number

4) Convert unit to dspacing on V-Nb and add each bank together to get 2 spectr.

5) Fit spline to 2 spectrum and store

6) Convert unit ceria to dspacing and divide each pixel with the corresponding curve

7) Divide each ceria pixel by corrected sensitivity number

Change History

comment:1 Changed 5 years ago by Lottie Greenwood

  • Description modified (diff)

comment:2 Changed 5 years ago by Lottie Greenwood

  • Description modified (diff)

comment:3 Changed 5 years ago by Lottie Greenwood

  • Status changed from new to assigned
  • Description modified (diff)

After testing, method 1 is found to be too noisy and not produce accurate results. So method 2 will be followed

Last edited 5 years ago by Lottie Greenwood (previous) (diff)

comment:4 Changed 5 years ago by Lottie Greenwood

  • Cc saurabh.kabra@… added
  • Description modified (diff)

From further discussion with EnginX Scientists, the scope of this ticket is expanded somewhat.

  • CalibrateFull needs to include loading and preparing vanadium for fitting, along with a calibration
  • Calibrate needs to do as above but save the two splines and sensitivities outputted from the vanadium calibration
  • EnginXFocus should be renamed as EnginXDetFocus and a new EnginXFocus should be created that takes Difc and Zero from EnginXCalibrate rather than Detector Positions

comment:5 Changed 5 years ago by Nick Draper

  • Milestone changed from Release 3.4 to Release 3.5

Moved to R3.5 at the R3.4 code freeze

comment:6 Changed 5 years ago by Lottie Greenwood

  • Owner changed from Lottie Greenwood to Federico M Pouzols

comment:7 Changed 5 years ago by Federico M Pouzols

  • Blocking 8368 added

(In #8368) Added new ENGIN-X tickets that have been opened in the last few months as 'blocked by'.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12426

Note: See TracTickets for help on using tickets.