Sorting in splatter plot point generation

[Peter Peterson]

Since the factory is a property of the splatter plot itself, cache the sorting of the box structure so when people play with the threshold the sorting is not repeated.

[Dennis Mikkelson]

This should be done and should be fairly straightforward. It was not done initially, since "premature optimization is the root of all evil".

[Dennis Mikkelson]

RE 7213_Sort once in splatterplot

Added instance variables to hold the sorted list of boxes and the workspace name. The list sorted list will now only be made if it was not previously made, or if the MD workspace name changes. NOTE: This does not yet solve the problem, since the whole splatter plot factory is rebuilt everytime the user changes the number of points or percent. One possible solution would be for the higher level class to only construct a new splatter plot factory in case the workspace changes, and just call the doCreate() method on that factory when the user changes the number of points or percent.

refs #7213

Changeset: 7771162aa815148df6188919e052366104f20469

[Dennis Mikkelson]

The change to keep a sorted list requires more extensive modification to the VSI, as described in the previous comment on this ticket. In addition, since the time when work was done on this ticket, the SplatterPlotFactory code has been changed to create VTK data structures that are precisely the correct size. Consequently, if the work in this branch is to be continued, together with more extensive changes to the higher level classes in the VSI, a thoughtful, manual merge of the changes here with the new code should be done.

[Nick Draper]

Moved to backlog at the code freeze for R2.6

[Michael Reuter]

This work has been translated into #7728. When that ticket is closed, you can close this one for free.

[Peter Peterson]

I verified the other and deleting both branches.

[Stuart Campbell]

This ticket has been transferred to github issue ​8059