Ticket #1616 (closed: worksforme)

Opened 10 years ago

Last modified 5 years ago

Problems displaying and load-saving workspaces created by Qxy

Reported by: Steve Williams Owned by: Steve Williams
Priority: major Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Martyn, Gigg Blocked By:
Blocking: Tester: Michael Whitty

Description

Workspaces created by a 2D SANS reduction have these problems

  • Right clicking on the workspace view and clicking "View Detectors" raises an unhandled exception
  • When the workspace is saved as nexus and then load re-loaded: The new workspace doesn't display correctly on 2D plot and ckWorkspacesMatch() against the original returns false.

Change History

comment:1 Changed 10 years ago by Steve Williams

  • Status changed from new to accepted

Time estimate:

  • Fix Qxy() 2 hours (including diagnosing the problem, fixing, extending the unit tests)
  • Extend NexusSaveProcessed() to deal with the unusual format of workspaces created by Qxy() 4 hours (elarn more about numeric axis and the nexus format and find a solution 1 hour. Implement the solution 2 hours. Write tests and debug 1 hour)
  • Extend NexusLoadProcessed() to load the new format and test 2 hours

Total 8 hours

comment:2 Changed 10 years ago by Martyn Gigg

M Gigg: As a general comment this should be mostly focussed on ensuring that Load/Save Nexus processed can deal with workspaces that don't have spectra axes anymore. Qxy is most probably correct it is just other parts of the code that generally expect spectra axes that need to be updated. Please let me know if this is not the case.

Time estimate: I think as a whole 1 day should be sufficient for this.

comment:3 Changed 10 years ago by Steve Williams

  • Component set to Mantid

Consider implementation at the same as working on ticket #1673

comment:4 Changed 10 years ago by Steve Williams

  • Milestone changed from Iteration 25 to Iteration 26

comment:5 Changed 10 years ago by Steve Williams

(In [8058]) Nexus needs Ws2_32.lib to be included, include it on Visual Studio 64-bit builds re #1616

comment:6 Changed 10 years ago by Steve Williams

(In [8074]) The "View Detector Table" command was raising an exception if there is no index-spectra map in the workspace, catch exception it re #1616

comment:7 Changed 10 years ago by Steve Williams

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

Most of the problems here had gone away. The two pieces of functionality listed at the top work now!

comment:8 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty

comment:9 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

indeed. "view detectors" on workspace without a spectra axis no longer raises an unhandled exception and saving/loading of non spectra axis was dealt with in #1993.

comment:10 Changed 5 years ago by Stuart Campbell

  • Cc Martyn, Gigg added; Martyn Gigg removed

This ticket has been transferred to github issue 2463

Note: See TracTickets for help on using tickets.