Ticket #5353 (closed: fixed)
LoadEventNexus with ISIS data & MonitorsAsEvents fails
| Reported by: | Martyn Gigg | Owned by: | Martyn Gigg |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.1.1 |
| Component: | Mantid | Keywords: | PatchCandidate,Released |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Roman Tolchenov |
Description
Running LoadEventNexus with an LET event file and MonitorsAsEvents set to True causes an error that "event_pixel_id" cannot be opened.
This is stopping reduction on LET
Change History
comment:2 Changed 8 years ago by Martyn Gigg
Add a system test for loading LET event monitors. Refs #5353
It takes far too long for a unit test to load a file with monitors as events so this is the next best thing.
Changeset: 5c456c6398f79631986e28381c053f5224c8d745
comment:3 Changed 8 years ago by Martyn Gigg
- Owner set to Martyn Gigg
- Status changed from new to accepted
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 Nick Draper
- Milestone changed from Release 2.2 to Release 2.1.1
Moved for inclusion in the 2.1.1 patch
comment:6 Changed 8 years ago by Martyn Gigg
Fix MonitorsAsEvents for ISIS event files. Refs #5353
The tweaks for performance didn't quite work for ISIS data. The spectrum numbers were wrong. I will add a test when I get some data that we can put in the repository. (cherry picked from commit 8aa4a06b679425bbbafe01329193b81b5797959d)
Changeset: 6abba812e2ed33d905e668495a60b1f480680790
comment:7 Changed 8 years ago by Roman Tolchenov
- Status changed from verify to verifying
- Tester set to Roman Tolchenov
comment:8 Changed 8 years ago by Roman Tolchenov
- Status changed from verifying to closed
Loaded an LET event file. Monitors are loaded into a separate event workspace.
comment:9 Changed 7 years ago by Nick Draper
- Keywords PatchCandidate,Released added; PatchCandidate removed
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 6199
Fix MonitorsAsEvents for ISIS event files. Refs #5353
The tweaks for performance didn't quite work for ISIS data. The spectrum numbers were wrong. I will add a test when I get some data that we can put in the repository.