Ticket #6382 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Goniometer parameter for TOPAZ

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

SetGoniometer should have an extra parameter with name goniometers (the only one I know already is "Universal" which is what TOPAZ currently uses). This will take the place of the other n-parameters and set the axes and motor names according to goniometers that we define in the code.

Change History

comment:1 Changed 8 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Vickie Lynch

Refs #6382 goniometers parameter for SetGoniometer

Changeset: f39da323baa32def4b23e84f25986506eb9db997

comment:3 Changed 8 years ago by Vickie Lynch

Refs #6382 catch for no logs and test added

Changeset: 8f949dc3bfea8d4e1e4a1591347ae471f0c0130d

comment:4 Changed 8 years ago by Vickie Lynch

  • Status changed from accepted to verify
  • Resolution set to fixed
Last edited 8 years ago by Vickie Lynch (previous) (diff)

comment:5 Changed 8 years ago by Vickie Lynch

To Test: 

LoadEmptyInstrument(Filename='TOPAZ_Definition_2012-08-23.xml',OutputWorkspace='topaz')
SetGoniometer(Workspace='topaz',Goniometers='Universal')
LoadLogsFromSNSNexus(Workspace='topaz',Filename='TOPAZ_6251_event.nxs')
SetGoniometer(Workspace='topaz',Goniometers='Universal')
print mtd[topaz].getRun().getGoniometer().getEulerAngles("yzy")
Last edited 8 years ago by Vickie Lynch (previous) (diff)

comment:6 Changed 8 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:7 Changed 8 years ago by Peter Peterson

  • Status changed from verifying to closed

This works correctly.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7228

Note: See TracTickets for help on using tickets.