Ticket #4580 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

NormaliseToMonitor option for MonitorWorkspace with more than 1 spectrum

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Shelly Ren

Description

TOPAZ monitor workspaces have downstream and upstream monitors so SNS needs option to choose which monitor for normalization.

Change History

comment:1 Changed 9 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Vickie Lynch

Refs #4580 MonitorWorkspaceSpectrum option

Changeset: f6e65733556c8a3d26597e1f143f738d0bf9bc81

comment:3 Changed 9 years ago by Vickie Lynch

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

comment:4 Changed 8 years ago by Shelly Ren

  • Status changed from verify to verifying
  • Tester set to Shelly Ren

comment:5 Changed 8 years ago by Shelly Ren

  • Status changed from verifying to closed

Tested TOPAZ run 3676. Ran loadEventNexus via MantidPlot with fileName = 'TOPAZ3676', OutputWorkspace='top',LoadMonitors='1',MonitorsAsEvents='1'. Then ran NormaliseToMonitor(InputWorkspace='top',OutputWorkspace='top_0',MonitorWorkspace='top_monitors') and NormaliseToMonitor(InputWorkspace='top',OutputWorkspace='top_1',MonitorSpectrum='1', MonitorWorkspace='top_monitors') respectively.

The difference between the two NormaliseToMonitor calls is the value of MontiorSpectrum. It was set to 0 in the first call, and 1 in the 2nd call. TOPAZ has 2 monitors but only the downstream monitor has data. Therefore, when I tried "show instruments" from the output workspace having the upstream monitor, the MantidPlot showed an error message "The workspace contains values that cannot be displayed" while it looked fine in the output workspace having the downstream monitor. With this fix, SNS has the option to choose which monitor for normalization.

comment:6 Changed 8 years ago by Shelly Ren

Vickie is going to discuss with Russell on another parameter MontiorWorkspaceIndex for the NormalisetoMonitor algorithm.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5427

Note: See TracTickets for help on using tickets.