Ticket #6773 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

GetEi needs to calculate the peak in the first monitor from the data regardless of whether Ei is fixed

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

Currently, if Ei is fixed, then GetEi use the distance between the detectors and the given energy value to calculate where the peak in the first monitor spectrum should be. If the neutron pulse is slim, i.e high energy neutrons, then this value is close to the true value from the monitor spectrum itself and taking this as t=0.0 is a good enough approximation.

If, as is the case on LET, that the energy is much lower then the width of the neutron pulse will result in a difference between the effective time zero and that calculated from the nominal fixed ei value.

To get the correct t=0 value the time of the first monitor should always be calculated from the peak in the first monitor spectrum.

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:3 Changed 8 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:4 Changed 8 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:5 Changed 8 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:6 Changed 8 years ago by Martyn Gigg

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

Branch: feature/feature/6773_calculate_peaktof_getei2

comment:7 Changed 8 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:8 Changed 8 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:9 Changed 8 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:10 Changed 8 years ago by Andrei Savici

  • Status changed from verifying to closed

Works as described, and it does not break if you put bad workspaces for HYSPEC (it calculates t0 from the formula). However, if you do it on a bad SEQUOIA workspace (event workspace for example) it gives an error. This is going to be fixed in ticket #6782

comment:11 Changed 8 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:12 Changed 8 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:13 Changed 8 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:14 Changed 8 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:15 Changed 7 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:16 Changed 7 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:17 Changed 7 years ago by Martyn Gigg

Fix GetEi2 to calculate monitor peak time from data

even for a fixed Ei. Refs #6773

Changeset: 76c69c496fe97e31a8f1dc66a840d752ff7eecee

comment:18 Changed 7 years ago by Martyn Gigg

Add a test for fixing ei. Refs #6773

Changeset: e008e2595f5eb9626df843096afb2ce745f34868

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7619

Note: See TracTickets for help on using tickets.