Ticket #4987 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Add ability to display either the neutronic or physical geometry in the instrument view window.

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Jose Borreguero

Description

This will probably be used more for debugging instrument geometry files than by real users (I think).

But it would be very useful to be able to view either the neutronic or physical positions for beamlines with both.

Change History

comment:1 Changed 9 years ago by Stuart Campbell

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Stuart Campbell

Added property to control which geometry is used. refs #4987

Added a check for the value of the instrument.view.geometry property. The default behaviour will be to view the main geometry.

Changeset: 206290d0ff13158464d140c9ea7d996de6a54485

comment:3 Changed 9 years ago by Stuart Campbell

Added instrument.view.geometry property into template. refs #4987

Added a default value for the property that controls which geometry is displayed in the instrument view window.

Changeset: c1c35ee11cdf74b65bef6cb23a33ca0431cc3f58

comment:4 Changed 9 years ago by Stuart Campbell

To test: 

 LoadEmptyInstrument(Filename=r'INDIRECT_Definition.xml',OutputWorkspace='w1')
 config['instrument.view.geometry'] = 'Physical'
 instrument_view = getInstrumentView('w1')
 instrument_view.show()
 config['instrument.view.geometry'] = 'Neutronic'
 instrument_view2 = getInstrumentView('w1')
 instrument_view2.show()

Uou should see two different geometry displayed.

The INDIRECT_Definition.xml is in the IDFs_for_UNIT_TESTING directory in the instrument folder.

Last edited 8 years ago by Stuart Campbell (previous) (diff)

comment:5 Changed 9 years ago by Stuart Campbell

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

comment:6 Changed 9 years ago by Stuart Campbell

Added property to control which geometry is used. refs #4987

Added a check for the value of the instrument.view.geometry property. The default behaviour will be to view the main geometry.

Changeset: 206290d0ff13158464d140c9ea7d996de6a54485

comment:7 Changed 9 years ago by Stuart Campbell

Added instrument.view.geometry property into template. refs #4987

Added a default value for the property that controls which geometry is displayed in the instrument view window.

Changeset: c1c35ee11cdf74b65bef6cb23a33ca0431cc3f58

comment:8 Changed 8 years ago by Jose Borreguero

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

comment:9 Changed 8 years ago by Jose Borreguero

  • Status changed from verifying to closed

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5833

Note: See TracTickets for help on using tickets.