Ticket #5001 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

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:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

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

comment:6 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

Yes, this is working.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5847

Note: See TracTickets for help on using tickets.