Ticket #10918 (assigned)

Opened 6 years ago

Last modified 5 years ago

BugFix: Nexus loader issue

Reported by: Anton Piccardo-Selg Owned by: Anton Piccardo-Selg
Priority: major Milestone: Release 3.5
Component: Framework Keywords:
Cc: Andrei, Savici Blocked By:
Blocking: Tester:

Description (last modified by Anton Piccardo-Selg) (diff)

PART 1: The ParaView loader plugins have an issue with loading MDEvent workspaces which are essentially 0D.

To reproduce:

  1. Create a sample data set using the attached python script. Apply SaveMD to save the data set.
  2. Set your path variables (for debug or release)

PV_PLUGIN_PATH: Mantid's ParaView plugins path for debug or release PATH: SOURCEPATH\Code\Third_Party\lib\win64

SOURCEPATH\Third_Party\lib\win64\Python27 BUILDPATH\bin\Debug or BUILDPATH\bin\Release

MANTIDPATH: BUILDPATH\bin\Debug or BUILDPATH\bin\Release

Note that SOURCEPATH and BUILDPATH need to be adapted to your specifc setting.

  1. Run ParaView (in debug or release)
  2. Load the created data set.

This should cause ParaView to abort.

Cause: The loader plugin checks for the dimensionality of the data set and passes the handling of it down a chain of responsibility if the data set does not match the dimensionality of the current data set factory. A 0D data set is not accounted for, hence the method castAndCheck of the class vtkDataSetFactory returns a NULL pointer as the workspace reference. The workspace is still marked as having a successor. Calling the doInitialize method on the workspace breaks the program.

PART 2: There is a related bug which occurs with the data set "setsymm_5_5_400_Steps.nxs". This data set has 3 non-integrated dimensions but a 4D data set factory gets invoked. This dimensionality mismatch creates a NULL reference for the workspace as above. The issue occurs at the exact same location in the code.

Attachments

paraViewTest.py​ (250 bytes) - added by Anton Piccardo-Selg 6 years ago.

Change History

Changed 6 years ago by Anton Piccardo-Selg

comment:1 Changed 6 years ago by Anton Piccardo-Selg

  • Description modified (diff)

comment:2 Changed 6 years ago by Anton Piccardo-Selg

  • Cc Andrei, Savici added
  • Description modified (diff)
  • Summary changed from BugFix: Nexus loader issue with 0D workspaces to BugFix: Nexus loader issue

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:4 Changed 5 years ago by Nick Draper

  • Milestone changed from Release 3.4 to Release 3.5

Moved to R3.5 at the R3.4 code freeze

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11757

Note: See TracTickets for help on using tickets.