Ticket #4306 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Line Viewer Crash 4 Dimensions

Reported by: Owen Arnold Owned by: Janik Zikovsky
Priority: minor Milestone: Release 2.0
Component: VATES Keywords:
Cc: Blocked By:
Blocking: Tester: Jose Borreguero

Description

file = MAR11001.raw #From data directory Load(Filename=file,OutputWorkspace='MAR11001') AddSampleLog(Workspace='MAR11001',LogName='phi',LogText='0',LogType='Number Series') AddSampleLog(Workspace='MAR11001',LogName='chi',LogText='0',LogType='Number Series') AddSampleLog(Workspace='MAR11001',LogName='omega',LogText='10',LogType='Number Series') SetUB(Workspace='MAR11001',UB='1,0,0,0,1,0,0,0,1') GetEi(InputWorkspace='MAR11001',Monitor1Spec='2',Monitor2Spec='3',EnergyEstimate='13') ConvertToMDEvents(InputWorkspace='MAR11001',OutputWorkspace='ss',QDimensions='QxQyQz',MinValues='-10,-10,-10,-2',MaxValues='10,10,10,10',u='1,0,0',v='0,1,0') BinMD(InputWorkspace='ss', AlignedDimX='Q_x,-10,10,200', AlignedDimY='Q_y,-10,10,200', AlignedDimZ='Q_z,-10,10,200', AlignedDimT='DeltaE,-2,10,100', OutputWorkspace='binned')

Now try and run the line viewer by selecting a line. Throws exception LineViewer::apply(). Too many dimensions. Which is correct, but we should handle this better in mantidplot. Some options are:

  • Disable Line view for 4+ dimensions
  • Raise Qt message box instead and don't get to the preview plot point. Catch the exception and make sure code is exception safe.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

Reproducing the problem - though the LineViewer should be able to handle 4+D for the inelastic people, so I will fix it to be able to apply.

comment:2 Changed 9 years ago by Janik Zikovsky

Refs #4306: Fix line integration in 4D

You can now use the LineViewer for up to 4D line integration. Will still fail at 5+D but this is a limitation of the BinMD/etc. algorithms. Will be easy to fix when needed

Changeset: 75bfa0d00b116d7e90b3127ebe598098c02fb00f

comment:3 Changed 9 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:4 Changed 9 years ago by Janik Zikovsky

Refs #4306: Fix line integration in 4D

You can now use the LineViewer for up to 4D line integration. Will still fail at 5+D but this is a limitation of the BinMD/etc. algorithms. Will be easy to fix when needed

Changeset: 75bfa0d00b116d7e90b3127ebe598098c02fb00f

comment:5 Changed 9 years ago by Jose Borreguero

  • Status changed from verify to verifying
  • Tester set to Jose Borreguero

comment:6 Changed 9 years ago by Jose Borreguero

  • Status changed from verifying to closed

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5153

Note: See TracTickets for help on using tickets.