Opened 11 years ago

Closed 10 years ago

#1261 closed feature request (done)

Automatic Coloring in BubbleChart

Reported by: mkommend Owned by: mkommend
Priority: highest Milestone: HeuristicLab 3.3.2
Component: Optimization.Views Version: 3.3.2
Keywords: Cc:

Description

It should be possible to color all runs automatically according to their y-axis value.

Change History (14)

comment:1 Changed 11 years ago by mkommend

  • Priority changed from high to highest

comment:2 Changed 10 years ago by mkommend

  • Status changed from new to accepted

Added automatic coloring to the RunCollectionBubbleChartView with r4799.

comment:3 Changed 10 years ago by mkommend

  • Owner changed from mkommend to gkronber
  • Status changed from accepted to reviewing

comment:4 Changed 10 years ago by swagner

  • Milestone changed from HeuristicLab x.x.x to HeuristicLab 3.3.2

comment:5 follow-up: Changed 10 years ago by mkommend

We should discuss if the code to calculate the color gradient should be moved in a static class in the HL.Common project. This code part is actually exactly the same and used in the RunCollectionBubbleChartView and the HeatMapView.

comment:6 in reply to: ↑ 5 Changed 10 years ago by swagner

Replying to mkommend:

We should discuss if the code to calculate the color gradient should be moved in a static class in the HL.Common project. This code part is actually exactly the same and used in the RunCollectionBubbleChartView and the HeatMapView.

Yes, that is a good idea. Please feel free to move the gradient code into HeuristicLab.Common.

comment:7 Changed 10 years ago by mkommend

  • Owner changed from gkronber to mkommend
  • Status changed from reviewing to assigned

comment:8 Changed 10 years ago by mkommend

  • Status changed from assigned to accepted

comment:9 Changed 10 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

Extracted ColorGradient in a separate class with r4808.

comment:10 Changed 10 years ago by swagner

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to assigned

Thanks. I reviewed the changes and found the following issue:

  • An exception is thrown (index out of range), if a categorical property with only a single value is selected on any axis (no jitter) and the color button for this axis is clicked.

Furthermore, I would suggest the following modification:

  • If jitter is added, the automatic coloring colors the runs according to their jittered values. In my opinion, only the original values should be considered.

comment:11 Changed 10 years ago by mkommend

  • Status changed from assigned to accepted

comment:12 Changed 10 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

Corrected and redesigned automatic coloring in RunCollectionBubbleChartView with r4845.

Last edited 10 years ago by swagner (previous) (diff)

comment:13 Changed 10 years ago by swagner

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to readytorelease

Great, thanks.

comment:14 Changed 10 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.1 to 3.3.2
Note: See TracTickets for help on using tickets.