Ticket #8736 (closed: invalid)

Opened 7 years ago

Last modified 5 years ago

Change detectors IDs

Reported by: Ricardo Leal Owned by: Ricardo Leal
Priority: major Milestone: Release 3.1
Component: Framework Keywords: Core
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

To append the instrument with other instrument / bank the detector IDs must be updated.

Change History

comment:1 Changed 7 years ago by Ricardo Leal

  • Status changed from new to inprogress

Re #8736 DetectorIds are now changeable.

Changeset: ce2b1659dcb5e43b281148cc67707ce33ca990c4

comment:2 Changed 7 years ago by Ricardo Leal

  • Keywords Core added
  • Status changed from inprogress to verify
  • Resolution set to fixed

Removed const from detId. Appended a few lines to testId().

comment:3 Changed 7 years ago by Russell Taylor

  • Status changed from verify to reopened
  • Resolution fixed deleted

I'm not too keen on this (see comments about immutability earlier today!). Comments are unfortunately not a reliable way to enforce behavior. It wouldn't surprise me to see this new method exposed to python by someone at some point in the future and then anyone could mess with it.

Could the goal be achieved by creating an entirely new Detector (and perhaps an entirely new instrument) with the correct ID. Two concerns come to mind: detector IDs are stored in the ISpectrum to map the data to the correct pixel. More importantly, instruments are stored and re-used in the Instrument Data Service and so modifying it behind the scenes could lead to surprises.

Either way the implementation of the method, however simple, should not be in the header file (even if someone else has done that).

comment:4 Changed 7 years ago by Ricardo Leal

There's better ways of doing this. Close invalid ticket.

comment:5 Changed 7 years ago by Ricardo Leal

  • Status changed from reopened to verify
  • Resolution set to invalid

comment:6 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:7 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

OK. Please delete the branch on github.

comment:8 Changed 7 years ago by Ricardo Leal

Done!

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9580

Note: See TracTickets for help on using tickets.