Opened 4 years ago

Closed 3 years ago

#2120 closed enhancement (done)

RunCollection doesn't update ResultNames or ParameterNames when items are added to a run afterwards

Reported by: abeham Owned by: abeham
Priority: medium Milestone: HeuristicLab 3.3.11
Component: Optimization Version: 3.3.9
Keywords: Cc:

Description

Run stores its results and parameters using a standard .Net Dictionary which are not observable. Because of this Run cannot notify RunCollection of a change. RunCollection however caches the parameter and result names for quick access in the view. When updating the Runs the caches in the RunCollection need to be updated as well. I would suggest to add a method to RunCollection which updates the cached properties and which can be called from whoever is updating the runs.

Change History (10)

comment:1 Changed 4 years ago by mkommend

Would it help to use ObserveableDictionary instead of the .Net Dictionary and update the cache in the RunCollection when a change occurs?

comment:2 Changed 4 years ago by gkronber

  • Owner changed from architects to abeham
  • Status changed from new to assigned

comment:3 Changed 4 years ago by abeham

  • Milestone changed from HeuristicLab 3.3.x Backlog to HeuristicLab 3.3.10

comment:4 Changed 3 years ago by abeham

  • Milestone changed from HeuristicLab 3.3.10 to HeuristicLab 3.3.11

comment:5 Changed 3 years ago by abeham

  • Status changed from assigned to accepted

comment:6 Changed 3 years ago by abeham

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

r11344:

  • Parameters and Results are now ObservableDictionaries
  • PropertyChanged event handler replaces the Changed event handler
  • RunCollection listens to changed events to each run's parameters and results (8 additional event handlers per run)

comment:7 Changed 3 years ago by abeham

r11345: fixed event handling

comment:8 Changed 3 years ago by mkommend

r11419: Changed access modifier for Run.Parameters setter from public to private.

comment:9 Changed 3 years ago by mkommend

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

Reviewed r11344 and r11345.

comment:10 Changed 3 years ago by abeham

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

r11522: merged to stable

Note: See TracTickets for help on using tickets.