Ticket #5001 (closed: fixed)
Change to LoadMD breaks ParaView MDEventNeXus file reader
Reported by: | Michael Reuter | Owned by: | Janik Zikovsky |
---|---|---|---|
Priority: | major | Milestone: | Release 2.1 |
Component: | VATES | Keywords: | |
Cc: | owen.arnold@… | Blocked By: | |
Blocking: | Tester: | Michael Reuter |
Description
The following changeset 76d074825f27294cf1bb6081628a8d988dc1cc18 broke the ability for the ParaView MDEventNeXus file reader plugins to actually read files. The error seen is
AnalysisDataService-[Warning] remove 'MD_EVENT_WS_ID' cannot be found LoadMD-[Notice] LoadMD started LoadMD-[Notice] LoadMD successful, Duration 0.00 seconds AnalysisDataService-[Warning] remove 'MD_EVENT_WS_ID' cannot be found LoadMD-[Notice] LoadMD started LoadMD-[Error] Error in execution of algorithm LoadMD LoadMD-[Error] NXopen(/home/2zr/git/mantid/Test/AutoTestData/SEQ_MDEW.nxs, 2) failed
This error happens no matter what type of MDNeXus file I try to open in ParaView or VatesSimpleGui.
I'm including Owen on the ticket in case this has to do with how the reader handles the FileBackEnd flag.
Change History
comment:2 Changed 9 years ago by Janik Zikovsky
Refs #5001: perhaps the nxs file was not getting closed properly
before being reopened in Read-Write mode. It seems that loading the meta data with 'FileBackEnd' set works well.
Changeset: 6f4c5b30053242aea4952fd32d92b0d40b685737
comment:3 Changed 9 years ago by Janik Zikovsky
- Status changed from accepted to verify
- Resolution set to fixed
I'm not 100% sure why it wasn't working (because the temporary workspace for loading metadata should close the file) but this seems to fix the error.
comment:4 Changed 9 years ago by Janik Zikovsky
Refs #5001: perhaps the nxs file was not getting closed properly
before being reopened in Read-Write mode. It seems that loading the meta data with 'FileBackEnd' set works well.
Changeset: 6f4c5b30053242aea4952fd32d92b0d40b685737
comment:5 Changed 8 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter