Ticket #5250 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Goniometer can not work with psi value loaded from nxspe file

Reported by: Alex Buts Owned by: Andrei Savici
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: saviciat@… Blocked By:
Blocking: Tester: Stuart Campbell

Description

goniometer expects time series property and psi is loaded as a number property.

additional enhancement, which needs discussion would be setting the goniometer into psi,0,1,0,1 position when loading nxspe file too.

Change History

comment:1 Changed 8 years ago by Andrei Savici

  • Owner set to Andrei Savici
  • Status changed from new to assigned

comment:2 Changed 8 years ago by Andrei Savici

  • Status changed from assigned to accepted

comment:3 Changed 8 years ago by Andrei Savici

Here is the test

Load(Filename='CNCS_7860_event.nxs',OutputWorkspace='in')
ConvertUnits(InputWorkspace='in',OutputWorkspace='in',Target='DeltaE',EMode='Direct',EFixed='3')
Rebin(InputWorkspace='in',OutputWorkspace='in',Params='-3,0.05,3')
SaveNXSPE(Filename='test.nxspe',InputWorkspace='in',Efixed='3',Psi='10')
LoadNXSPE(Filename='test.nxspe',OutputWorkspace='out')
SaveNexus(Filename='test.nxs',InputWorkspace='out')

Use hdfview to inspect test.nxs. It should have a goniometer, num_axes=1, and in axis 0: name='psi', angle='10',rotationaxis='0,1,0'

comment:4 Changed 8 years ago by Andrei Savici

Added goniometer on LoadNXSPE. Refs #5250

Allow run object to set goniometer on PropertyWithValue, not only TimeSeriesProperty

Changeset: 5ba68c7e9f8779bceda9a395830d16faa46dd5f4

comment:5 Changed 8 years ago by Andrei Savici

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

comment:6 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:7 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

Shouldn't the units be degree rather than deg ?

Also, there isn't a nexus class called NXmotor - that is what NXpositioner is for.

But these are minor side issues! :-)

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6096

Note: See TracTickets for help on using tickets.