Ticket #8650 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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

TOPAZ_mon_distance.jpeg (119.2 KB) - added by Peter Peterson 7 years ago.

Change History

Changed 7 years ago by Peter Peterson

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

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9494

Note: See TracTickets for help on using tickets.