Ticket #7362 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Allow user to specify detector ID in EditInstrumentGeometry

Reported by: Wenduo Zhou Owned by: Peter Peterson
Priority: critical Milestone: Release 2.6
Component: Framework Keywords:
Cc: petersonpf@… Blocked By:
Blocking: #7336, #7364 Tester: Russell Taylor

Description (last modified by Wenduo Zhou) (diff)

Currently EditInstrumentGeometry generates a detector ID for each spectrum by its own decision. Add a property such that user can specify the detector IDs of each spectrum.

Attachments

ticket7362.py (1.1 KB) - added by Peter Peterson 7 years ago.

Change History

comment:1 Changed 7 years ago by Wenduo Zhou

Fixed the issue. Refs #7362.

Changeset: dcc47151c187f08de8f2ad470d6e9cdcd5b1596a

comment:2 Changed 7 years ago by Wenduo Zhou

  • Description modified (diff)

comment:3 Changed 7 years ago by Wenduo Zhou

  • Status changed from new to accepted

comment:4 Changed 7 years ago by Wenduo Zhou

  • Description modified (diff)

comment:5 Changed 7 years ago by Wenduo Zhou

How to verify

  1. Locate data file PG3_4844_event.nxs.
  2. Download script ticket7362.py.
  3. Run the script in Mantid. With user specified detector IDs, the high resolution TOF workspace and low resolution TOF workspace can be conjoined.

comment:6 Changed 7 years ago by Wenduo Zhou

  • Blocking 7364 added

comment:7 Changed 7 years ago by Peter Peterson

  • Owner changed from Wenduo Zhou to Peter Peterson

Changed 7 years ago by Peter Peterson

comment:8 Changed 7 years ago by Peter Peterson

Re #7362. Added validateInputs to the top.

Also re-arranged a bunch of other code. Part of the "clean-up" that Wenduo had mentioned.

Changeset: a21387bb9358dace0841086d46ff7f41634fc7e1

comment:9 Changed 7 years ago by Peter Peterson

Re #7362. Finished making some parameters optional.

Changeset: 1bd6091dfcb316e1f393da3f891eaba1f1abbfa4

comment:10 Changed 7 years ago by Wenduo Zhou

Checkpointing work. Refs #7362.

  1. Continue to cleaning codes by removing some legacy which is not used

now;

  1. Removing properties NewInstrument by merging it to InstrumentName

Changeset: 26800470ef072f7a19066b19c588fd17f9ceb7a4

comment:11 Changed 7 years ago by Wenduo Zhou

Finished cleaning codes. Refs #7362.

Changeset: 235c751feaa0bab52ff35a7b9d1a1e26afba709a

comment:12 Changed 7 years ago by Wenduo Zhou

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

comment:13 Changed 7 years ago by Russell Taylor

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

comment:14 Changed 7 years ago by Wenduo Zhou

Fixed some possible memory leakages. Refs #7362.

Changeset: 0323c0e3def991e99800bac130ed2a152b144221

comment:15 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

comment:16 Changed 7 years ago by Russell Taylor

Oh dear - this broke a system test because of the removal of the 'NewInstrument' property. See #7402.

comment:17 Changed 7 years ago by Andrei Savici

Wiki still contains reference to NewInstrument

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8208

Note: See TracTickets for help on using tickets.