Ticket #5946 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

LoadRaw on multi-period data with separate monitors and SpectrumList=monitorID is incorrect

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description (last modified by Martyn Gigg) (diff)

Running this script gives an incorrect workspace list and also the ADS does not contain the correct workspaces

LoadRaw(Filename=r'CSP78173.raw',OutputWorkspace='CSP78173',SpectrumList='2',LoadMonitors='Separate')

print mtd.getObjectNames()

You should just get a group with the name 'CSP78173' and no separate monitors workspace because all you have asked to be loaded is the monitors so there is nothing to separate them from

Attachments

LoadRawExcludeMonitorsBug.png (212.4 KB) - added by Martyn Gigg 8 years ago.

Change History

Changed 8 years ago by Martyn Gigg

comment:1 Changed 8 years ago by Martyn Gigg

  • Owner set to Martyn Gigg
  • Status changed from new to accepted
  • Description modified (diff)
  • Summary changed from LoadRaw on multi-period data with separate monitors displays incorrectly to LoadRaw on multi-period data with separate monitors and SpectrumList=monitorID is incorrect

comment:2 Changed 8 years ago by Martyn Gigg

Formatting changes only. Refs #5946

Changeset: dcc886eeb11548bf1c41a5dabed9a881eae5c6db

comment:3 Changed 8 years ago by Martyn Gigg

Fix LoadRaw for multi-period & single monitor spectra... Refs #5946

with LoadMonitors=Separate.

Changeset: 68695b510f80c7a33b9c23f060a6b52e226b3bff

comment:4 Changed 8 years ago by Martyn Gigg

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

comment:5 Changed 8 years ago by Owen Arnold

refs #5946. Alternative histogramming.

Add the ability to histogram the event list by pulse_time

Changeset: 4d6a2a5991ad702303971a760e0a71a0b873de71

comment:6 Changed 8 years ago by Owen Arnold

Sorry, the above commit should have gone against #5948

comment:7 Changed 8 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:8 Changed 8 years ago by Karl Palmen

  • Status changed from verifying to closed

One group of name CSP78173 is observed with 12 members and no separate workspace for monitors.

comment:9 Changed 8 years ago by Martyn Gigg

Formatting changes only. Refs #5946

Changeset: dcc886eeb11548bf1c41a5dabed9a881eae5c6db

comment:10 Changed 8 years ago by Martyn Gigg

Fix LoadRaw for multi-period & single monitor spectra... Refs #5946

with LoadMonitors=Separate.

Changeset: 68695b510f80c7a33b9c23f060a6b52e226b3bff

comment:11 Changed 8 years ago by Owen Arnold

refs #5946. Alternative histogramming.

Add the ability to histogram the event list by pulse_time

Changeset: 4d6a2a5991ad702303971a760e0a71a0b873de71

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6792

Note: See TracTickets for help on using tickets.