Review implementation of 'contour + color fill' plotting option

[mantid-builder]

This issue was originally TRAC 3833

Original Reporter: Russell Taylor

This follows on from http://trac.mantidproject.org/mantid/ticket/3832 & http://trac.mantidproject.org/mantid/ticket/3240. We should check whether there are other places where checks for the Graph::ColorMapContour curve type should be added (should look at all the places where Graph::ColorMap & Graph::Contour appear).

Actually, it's not obvious to me that adding this extra curve type was the right thing to do. Isn't this actually 2 curves - a colorfill & a contour? I ask because I find that although I can switch the contour lines on and off in this type of plot, I can't remove the color map - even though the 'Plot Details' dialog suggests that I ought to be able to. (N.B. It's the same situation with both the 'Color Fill' & 'Contour Lines' plot types: contour lines can be toggled, but not the color fill).

Also, why is it referred to as a 'UserHelperFunction' in the 'Plot Details' dialog? That doesn't seem a description that is helpful to the user.

[mantid-builder]

@NickDraper (2012-01-09T09:43:48):
Moved to iteration 33 at iteration 32 code freeze

---

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

---

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

---

@NickDraper (2012-10-28T11:39:25):
Moved to milestone 2.4

---

@NickDraper (2013-01-28T09:23:30):
Moved at the code freeze for release 2.4

---

@NickDraper (2013-04-29T09:50:05):
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