Opened 11 years ago
Closed 10 years ago
#2121 closed defect (done)
Bubble chart view throws exception on hidden categories
Reported by: | abeham | Owned by: | mkommend |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.10 |
Component: | Optimization.Views | Version: | 3.3.9 |
Keywords: | Cc: |
Description
In AddDataPoint the check if a run is visible would need to be done before getting the value. Categorical values that are not visible are not mapped and therefore throw a KeyNotFoundException.
The building of the categorical mapping excludes invisible runs in the Where condition (GetCategoricalValue).
Change History (10)
comment:1 Changed 11 years ago by abeham
- Owner changed from swagner to abeham
- Status changed from new to accepted
comment:2 Changed 11 years ago by abeham
- Owner changed from abeham to mkommend
- Status changed from accepted to assigned
comment:3 Changed 11 years ago by mkommend
- Status changed from assigned to accepted
comment:4 Changed 10 years ago by mkommend
r11007: Fixed a bug in the bubble chart and boxplot view regarding filtered runs.
Edit: you must use the revision number in ticket commets, not the ticket number
comment:5 Changed 10 years ago by mkommend
- Owner changed from mkommend to abeham
- Status changed from accepted to reviewing
Please test the functionality of this ticket extensively as the interplay between filtering and displaying runs is rather complex.
comment:6 Changed 10 years ago by abeham
- Owner changed from abeham to mkommend
- Status changed from reviewing to assigned
In a running experiment where runs are constantly added the bubble chart updates the axis description upon mouseover in the axis combobox. The boxplot crashes.
comment:7 Changed 10 years ago by mkommend
- Owner changed from mkommend to abeham
- Status changed from assigned to reviewing
r11024: Fixed bugs in the bubble chart and box plot view regarding the axis labeling.
comment:8 Changed 10 years ago by abeham
- Owner changed from abeham to mkommend
I tested the changes and found no further bugs. I also looked at the source changes and only found a small typo: in the comment it should be "hovering"
Please fix the comment and release the changes
comment:9 Changed 10 years ago by gkronber
- Status changed from reviewing to readytorelease
r11095: fixed the comment as suggested by abeham.
comment:10 Changed 10 years ago by mkommend
- Resolution set to done
- Status changed from readytorelease to closed
This is tricky