Ticket #6380 (closed: fixed)
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
Change History
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