Opened 4 years ago

Closed 3 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 4 years ago by abeham

  • Owner changed from swagner to abeham
  • Status changed from new to accepted

comment:2 Changed 4 years ago by abeham

  • Owner changed from abeham to mkommend
  • Status changed from accepted to assigned

This is tricky

comment:3 Changed 4 years ago by mkommend

  • Status changed from assigned to accepted

comment:4 Changed 3 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

Last edited 3 years ago by abeham (previous) (diff)

comment:5 Changed 3 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 3 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 3 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 3 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 3 years ago by gkronber

  • Status changed from reviewing to readytorelease

r11095: fixed the comment as suggested by abeham.

comment:10 Changed 3 years ago by mkommend

  • Resolution set to done
  • Status changed from readytorelease to closed

r11139: Merged r11007, r11024, r11095 into stable.

Note: See TracTickets for help on using tickets.