Ticket #1513 (closed: fixed)
MergeRuns for EventWorkspaces
Reported by: | Janik Zikovsky | Owned by: | Janik Zikovsky |
---|---|---|---|
Priority: | major | Milestone: | Iteration 25 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Vickie Lynch |
Description
... should add event lists together, rather than create a workspace 2D out of it.
Change History
comment:1 Changed 10 years ago by Janik Zikovsky
- Milestone changed from Iteration 24 to Iteration 25
comment:4 Changed 10 years ago by Janik Zikovsky
- Status changed from new to accepted
forgot to accept
comment:5 Changed 10 years ago by Janik Zikovsky
(In [6100]) Refs #1605 and #1595 in a way. Made spectrum numbers and workspace indices match (simple 1:1 map) for EventWorkspaces. Added tracking of detector ID in EVentList. Operators (+) do a union of detector IDs set. SpectraDetectorMap is generated from the EventList detector IDs. Renamed functions to clarify getEventList and getEventListAtWorkspaceIndex. New functions for finalizing the creation of EventWorkspaces. Fixed DiffractionFocussing and ConjoinWorkspaces to use the new functionality - code is made a lot clearer. This work supports Refs #1513 and #1581.
comment:9 Changed 10 years ago by Janik Zikovsky
(In [6127]) Refs #1513: Added test with real CNCS data. Much improved speed for mismatched workspace indices, but only for 1 detector per spectrum; this could still take forever if the indices are mismatched, there are lots of spectra, and not all have one detector. This should be a very unusual occurrence (can't think of a way to do this).
comment:10 Changed 10 years ago by Janik Zikovsky
comment:11 Changed 10 years ago by Janik Zikovsky
- Status changed from accepted to verify
- Resolution set to fixed
comment:12 Changed 10 years ago by Vickie Lynch
- Status changed from verify to verifying
- Component set to Mantid
- Tester set to Vickie Lynch
comment:13 Changed 10 years ago by Vickie Lynch
- Status changed from verifying to closed
Merged event workspaces for SNAP prenexus files for 4111 and merged workspace is the same as the event nexus file for the same experiment.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 2360