Ticket #6380 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

LoadVesuvio: The normalization looks incorrect

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

The counts/us look far to high from LoadVesuvio. The shape seems okay so it is most probably an incorrect normalization procedure.

The attached file shows the sort of scale that is expected.

Attachments

vesuvio.dat (21.6 KB) - added by Martyn Gigg 8 years ago.

Change History

Changed 8 years ago by Martyn Gigg

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 silly error in Vesuvio normalisation. Refs #6380

Changeset: 1dc385cae217441f33c3bebf554731e02ca3bea6

comment:3 Changed 8 years ago by Martyn Gigg

Update LoadVesuvio system test after fixing normalisation issue.

Refs #6380

Changeset: 636a96c32357381d0d95ba4010ad5be48ce4be14

comment:4 Changed 8 years ago by Martyn Gigg

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

Tester: There are some vesuvio data files in the system test data directory, add this to your datasearch path. Use the LoadVesuvio algorithm with the following options:

  • RunNumbers=14188
  • Spectra=135
  • DifferenceType=Single

The Y axis should go up to about 0.5 not about 2e6 as it was doing. Also check for summed files, i.e change

  • RunNumbers=14188-14190

The Y axis should be on the order of ~1 for most options.

comment:5 Changed 8 years ago by Martyn Gigg

Remove old LoadEVSRaw that was never used. Refs #6380

Changeset: 33dd327cc969eeb97bbb810368fa53939d105ea0

comment:6 Changed 8 years ago by Martyn Gigg

Fix RetrieveRunInfo to not depend on old LoadEVSRaw. Refs #6380

It was using some helper methods/classes not the actual algorithm

Changeset: c9419e263dbedc1acf2e47c7bdd33f5a4e37c33a

comment:7 Changed 8 years ago by Martyn Gigg

Fix silly error in Vesuvio normalisation. Refs #6380

Changeset: 6cd1308cf0c547762742edf77a24f34baa77a959

comment:8 Changed 8 years ago by Martyn Gigg

Remove old LoadEVSRaw that was never used. Refs #6380

Changeset: fce4c0e37aed3bd1f447282119d8ac17fce1667f

comment:9 Changed 8 years ago by Martyn Gigg

Fix RetrieveRunInfo to not depend on old LoadEVSRaw. Refs #6380

It was using some helper methods/classes not the actual algorithm

Changeset: 77798c69ed7222ff0f85732347981ee1f8b1feec

comment:10 Changed 8 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:11 Changed 8 years ago by Nick Draper

  • Status changed from verifying to closed

tested for single files and ranges

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7226

Note: See TracTickets for help on using tickets.