ParameterMap: Speed up the way masked detectors are recorded (?)

[mantid-builder]

This issue was originally TRAC 4759

Original Reporter: @jzikovsky

Instead of using a "masked" string with key = detector pointer, I suggest that we record masked detectors in the ParameterMap with a simple map:

• Key = detector ID (not the component ID)
• Value = true if Masked, false if not masked.

Or, even faster would be a vector, which can be accessed in O(1) time instead of O(log(N)) time.

A third option is to hold a std::set<> of masked detector IDs. Looking them up will be very fast when few detectors are masked, and at worst O(log(N)) if most detectors are masked. It is also safer than the vector (no going out of bounds).

This should speed up instrument view (and others) significantly. Plotting TOPAZ takes 1000 msec + each time it redraws. 800 msec or so are spent looking for masked detectors.

[mantid-builder]

@NickDraper (2012-04-30T14:15:47):
Moved at end of release 2.1

---

@NickDraper (2012-08-10T12:45:13):
http://www.mantidproject.org/Release_Notes_2_2_Full_List

---

@NickDraper (2012-10-28T11:42:52):
Moved to release 2.4

---

@NickDraper (2013-04-29T09:50:06):
Moved to r2.6 at the end of r2.5

---

@NickDraper (2013-07-26T13:59:10):
Moved to the Backlog after the code freeze for R2.6

---

@NickDraper (2014-02-14T11:07:57):
bulk move to assigned at the into of the triage step

[NickDraper]

archived