Ticket #5384 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Mantid crashes when retrieving position of a moved detector

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

Description (last modified by Karl Palmen) (diff)

The attached python file (DetectorMove.py) causes Mantid to crash on line indicated.

Attachments

DetectorMove.py (647 bytes) - added by Karl Palmen 8 years ago.
Python script loads a MERLIN file and moves one of its detectors.
DetectorMoveWorkaround.py (703 bytes) - added by Karl Palmen 8 years ago.

Change History

Changed 8 years ago by Karl Palmen

Python script loads a MERLIN file and moves one of its detectors.

comment:1 Changed 8 years ago by Karl Palmen

  • Cc anders.markvardsen@… added

Changed 8 years ago by Karl Palmen

comment:2 Changed 8 years ago by Karl Palmen

  • Description modified (diff)

Getting the moved detector and then getting its position, as shown in the attached DetectorMoveWorkaround.py avoids the crash.

comment:3 Changed 8 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Martyn Gigg

Can anything sensible be done here?

comment:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:5 Changed 8 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:6 Changed 8 years ago by Martyn Gigg

Modifying original ParameterMap instead of copy. Refs #5384

A detector retrieved before the Move/Rotate can then see the updated parameters without retrieval.

Changeset: f2877833418597541d4acfff756b51cefaddeb36

comment:7 Changed 8 years ago by Martyn Gigg

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

comment:8 Changed 8 years ago by Martyn Gigg

Modifying original ParameterMap instead of copy. Refs #5384

A detector retrieved before the Move/Rotate can then see the updated parameters without retrieval.

Changeset: f2877833418597541d4acfff756b51cefaddeb36

comment:9 Changed 8 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:10 Changed 8 years ago by Karl Palmen

  • Status changed from verifying to closed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6230

Note: See TracTickets for help on using tickets.