RunCollectionBubbleChart improvements

[mkommend]

• Categorical values (e.g., algorithm name) shall be sorted alphabetically (also applies for the BoxPlots)
• Automatic coloring for large run collections must be faster
• Selection of runs to apply specific actions

[mkommend]

r9235: Implemented changes in BubbleChart and BoxPlots to improve the coloring and the display of categorical values.

[sforsten]

r9236:

• Content.UpdateOfRunsInProgress is set to true while updating the colors of the data points in RunCollectionBubbleChartView
• removed a useless line in RunCollectionBoxPlotView

[ascheibe]

Thanks, looks good!

[mkommend]

r9312: Changes in the bubblechart:

• Added possibility to select runs.
• Added option to hide all selected runs.
• Performance improvements regarding coloring, filtering and selection.
• Corrected minor bug regarding categorical values in the bubblechart and boxplot view.

Remark: All changesets except r9312 have been already been reviewed.

[mkommend]

r9313: Corrected text alignment of the newly introduced buttons in the bubblechart.

[sforsten]

r9340: fixed bug: an exception was thrown, when a bubblechart was open and the runs have been cleared, because UpdateMarkerSizes was called. A check was added if points are available in the diagram.

[sforsten]

ToDo:

• Selection of mutliple runs when holding "Ctrl"
• improve color selection
• add "Unhide all" and "Color reset" option in context menu
• increase default bubble size
• restructure context menu
• fix bugs in BoxPlot: if runs in BubbleChart are hidden, the labels in BoxPlot are incorrect.

Maybe selection should be persisted or a property selection should be added to runs

[mkommend]

r9435: Added patch from sforsten, that improves the BubbleChart- and BoxPlotView, that implements all previously mentioned comments.

[abeham]

r9441: Changed the LogTransform for transparency to a steeper curve. ln(x+1)/ln(2.0) is still almost linear for x in [0;1]. I changed this to (1-0.05^x)/0.95.

Remaining points:

• The private variable transperencyExponent is not used anywhere (and spelled wrong)
• The private variable runToHide is also not used anywhere
• The method HideRuns takes an IEnumerable<IRun> that is not used, it iterates over selectedRuns instead
• run.Color = Color.Black; in line 830: Color.Black should be a constant or static readonly field
• point.Color = Color.Red; in line 173: Color.Red should be a constant or static readonly field
• In method GetValue(IRun, SizeDimension) the first parameter is not used.

[sforsten]

r9447: added toggle of selection for single data points

[mkommend]

r9448: Removed unused code in the BubbleChartView.

[mkommend]

Review comments:

• The private variable transperencyExponent is not used anywhere (and spelled wrong)
• The private variable runToHide is also not used anywhere
• The method HideRuns takes an IEnumerable<IRun> that is not used, it iterates over selectedRuns instead

The three review comments above were addressed in r9448.

• run.Color = Color.Black; in line 830: Color.Black should be a constant or static readonly field
• point.Color = Color.Red; in line 173: Color.Red should be a constant or static readonly field

I decided against static fields regarding the default values for the run color, because this should be a property of the Run itself and furthermore there should be more general visual styles implemented. This will be addressed in another ticket / release.

• In method GetValue(IRun, SizeDimension) the first parameter is not used.

This is due to consistency reasons with the other GetValue methods.

[abeham]

ok