Ticket #3833 (assigned)
Review implementation of 'contour + color fill' plotting option
Reported by: | Russell Taylor | Owned by: | Anyone |
---|---|---|---|
Priority: | minor | Milestone: | Backlog |
Component: | MantidPlot | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: |
Description
This follows on from #3832 & #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.
Change History
comment:1 Changed 9 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Anyone
comment:2 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:3 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:5 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:6 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:9 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to the Backlog after the code freeze for R2.6
comment:10 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
bulk move to assigned at the into of the triage step
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4680