Ticket #8650 (closed: fixed)
TOPAZ monitor position
Reported by: | Andrei Savici | Owned by: | Peter Peterson |
---|---|---|---|
Priority: | major | Milestone: | Release 3.1 |
Component: | Diffraction | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Andrei Savici |
Description
The monitor before the sample should be at about -2.57m instead of -4.45. To test:
LoadNexusMonitors(Filename='TOPAZ_7892_event.nxs',OutputWorkspace='TM') ConvertUnits(InputWorkspace='TM',OutputWorkspace='TMOrigK',Target='Momentum') Rebin(InputWorkspace='TMOrigK',OutputWorkspace='TMOrigK',Params='0.2,0.01,5',PreserveEvents='0') MoveInstrumentComponent(Workspace='TM',ComponentName='monitor1',Z='-2.57',RelativePosition='0') ConvertUnits(InputWorkspace='TM',OutputWorkspace='TMNewK',Target='Momentum') Rebin(InputWorkspace='TMNewK',OutputWorkspace='TMNewK',Params='0.2,0.01,5',PreserveEvents='0') Load(Filename='TOPAZ_7892_event.nxs',OutputWorkspace='TOPAZ_7892_event') ConvertUnits(InputWorkspace='TOPAZ_7892_event',OutputWorkspace='TOPAZ_7892_event',Target='Momentum') Rebin(InputWorkspace='TOPAZ_7892_event',OutputWorkspace='TOPAZ_7892_event',Params='0.2,0.01,5',PreserveEvents='0') SumSpectra(InputWorkspace='TOPAZ_7892_event',OutputWorkspace='sum')
Spectra for monitor 1,2, and the sum over all detectors should start at the same position in k (or lambda). Ignore the prompt pulse, that should be in different position.
Attachments
Change History
comment:1 Changed 7 years ago by Peter Peterson
Xioping did a different calculation/calibration that puts it at 2.488m (engineering is 2.55m).
comment:2 Changed 7 years ago by Peter Peterson
- Status changed from new to inprogress
Re #8650. Moving TOPAZ monitor for all versions of the instrument.
Changeset: 57bb28ccc1db4080b1cf386e308a8d67c422af8c
comment:3 Changed 7 years ago by Peter Peterson
- Status changed from inprogress to verify
- Resolution set to fixed
This is on bugfix/8650_topaz_mon_distance.
To test: This is really just looking at the source code changes. The monitor at 4.45m should now be at 2.488m.
comment:4 Changed 7 years ago by Andrei Savici
- Status changed from verify to verifying
- Tester set to Andrei Savici
comment:5 Changed 7 years ago by Peter Peterson
Re #8650. Moving TOPAZ monitor for all versions of the instrument.
Changeset: cdf18ba897caddfa837a3df5285f53125face962
comment:6 Changed 7 years ago by Andrei Savici
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/bugfix/8650_topaz_mon_distance'
Full changeset: 0d6cc28557ec106146ff1ab016c7965159eff5f3