Ticket #928 (closed: fixed)

Opened 11 years ago

Last modified 5 years ago

LoadRaw: Add an Parameter to control the loading of monitors

Reported by: Nick Draper Owned by: Sofia Antony
Priority: critical Milestone: Iteration 20
Component: Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

Add an LoadMonitors Parameter to allow the following:

  1. Include (default)
  2. Exclude
  3. Separate

If the user selects Separate these should then be put into a separate workspace (or workspace group) with the name [OutputWorkspace]_monitors

Change History

comment:1 Changed 11 years ago by Nick Draper

  • Priority changed from major to critical

comment:2 Changed 11 years ago by Sofia Antony

(In [3337]) re #928 Added optional parameter to control the loading of monitors and creates separate workspace/workspace group for monitors.

comment:3 Changed 11 years ago by Sofia Antony

(In [3338]) re #928- Doxygen warnings fixed

comment:4 Changed 11 years ago by Sofia Antony

(In [3342]) re #928 -Changed the monitor workspace name to OutputWorkspace_Monitors.MantidQt files changed for fixing LoadRaw V.1 crash

comment:5 Changed 11 years ago by Sofia Antony

(In [3343]) re #928 -fix for test build failure

comment:6 Changed 11 years ago by Sofia Antony

(In [3354]) re#928 -fix for filling SpectraToDetectorMap when separate/exclude options selected from LoadRaw

comment:7 Changed 11 years ago by Sofia Antony

  • Status changed from new to accepted

comment:8 Changed 11 years ago by Sofia Antony

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

comment:9 Changed 11 years ago by Russell Taylor

  • Status changed from testing to reopened
  • Resolution fixed deleted

comment:10 Changed 11 years ago by Russell Taylor

(In [3492]) New MonitorList property on LoadInstrument should be an output property. Re #928.

comment:11 Changed 11 years ago by Russell Taylor

  • Status changed from reopened to closed
  • Resolution set to fixed

comment:12 Changed 11 years ago by Russell Taylor

Went straight from reopened to closed this time, not via 'testing'. Still, it's fine, including when the monitors are in the middle of the data.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​1775

Note: See TracTickets for help on using tickets.