Ticket #7967 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Use Locations tag to define all tube types in Merlin IDF

Reported by: Karl Palmen Owned by: Karl Palmen
Priority: major Milestone: Release 3.1
Component: Framework Keywords: maintenance
Cc: anders.markvardsen@… Blocked By:
Blocking: Tester: Martyn Gigg

Description

The Merlin IDF already uses the locations tag of one tube type, but still uses location tag for another tube type, which is consequently much longer. Ensure all tube types use the locations tag to make them compact.

Change History

comment:1 Changed 7 years ago by Karl Palmen

  • Owner set to Karl Palmen
  • Cc anders.markvardsen@… added

comment:2 Changed 7 years ago by Karl Palmen

  • Milestone changed from Backlog to Release 3.1

comment:3 Changed 7 years ago by Karl Palmen

  • Status changed from new to inprogress

comment:4 Changed 7 years ago by Karl Palmen

The IDF with the locations tag for one tube type did not make it to release, but I have a copy available and I'll take the tube definition (and nothing else) from there.

comment:5 Changed 7 years ago by Karl Palmen

Modify MERLIN_Definition.xml to use locations tag for tubes re #7967

This shortens the definition file considerably, but does increase loading time.

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: dd393761f51538560573b77951fdfede09ed8fa8

comment:6 Changed 7 years ago by Karl Palmen

To test, save a copy of the MERLIN_definition.xml file from the mantid instrument folder to one of your own folders (not a Mantid default folder) before running the git macros.

Then having run git test start, check that the tube definitions in the instrument file has been shortened by means of the locations tag. Check that both the new and old definition files load and the instrument has identical appearrence in both.

You may then load some MERLIN data files to check they are OK.

comment:7 Changed 7 years ago by Karl Palmen

Correct pixel type for 2nd type of tube re #7967

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 464eb5bcbe36c3a60192fedf21bacffbe47d427f

comment:8 Changed 7 years ago by Karl Palmen

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

comment:9 Changed 7 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:10 Changed 7 years ago by Martyn Gigg

  • Status changed from verifying to closed

Checked the random detectors in the detector list & the values look the same.

The system tests are also okay so that must mean the positions are correct to within an acceptable precision.

comment:11 Changed 7 years ago by Martyn Gigg

Merge remote-tracking branch 'origin/feature/7967_merlin_locations_tag'

Full changeset: 7194dbfdef8014991b89bb98b7bed420bf8f2b7f

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8812

Note: See TracTickets for help on using tickets.