Ticket #1728 (closed: wontfix)

Opened 10 years ago

Last modified 5 years ago

3D Instrument view: Refactor the way color for each detector is calculated

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

... will be using a map with key = detector ID and value = GLColor to use. This avoids the trickiness of the list of detector IDs having to be in a particular order, etc, and feels cleaner.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

(In [6794]) Refs #1728: Flag for getDetectorIDtoWorkspaceIndexMap() function.

comment:3 Changed 10 years ago by Janik Zikovsky

  • Priority changed from major to minor
  • Milestone changed from Iteration 25 to Iteration 26

I am not certain if there is a better way to handle this. Some methods would be cleaner, code-wise, but everything I found would run more slowly, which is to be avoided. Pushing to later milestone, if ever.

comment:4 Changed 10 years ago by Janik Zikovsky

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

Timing tests showed that while this step is not terribly fast, it is of the order of < 0.1 seconds IIRC, meaning that it is good enough even for fairly large instruments. Wont fix.

comment:5 Changed 10 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:6 Changed 10 years ago by Nick Draper

  • Status changed from verifying to closed

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2575

Note: See TracTickets for help on using tickets.