Opened 8 years ago
Closed 7 years ago
#2758 closed enhancement (done)
RunCollectionView is too slow
Reported by: | mkommend | Owned by: | mkommend |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.15 |
Component: | Optimization.Views | Version: | 3.3.14 |
Keywords: | Cc: |
Description
I recently create run collection with over 100K runs. With the current implementation of the RunCollectionView it is impossible to work with such run collections, although the table view or bubble chart handle those amounts just fine.
Therefore, the RunCollectionView must be sped up. For testing purpose I have attached a run collection with ~ 10,000 runs.
Attachments (1)
Change History (6)
Changed 8 years ago by mkommend
comment:1 Changed 8 years ago by mkommend
- Status changed from new to accepted
comment:2 Changed 8 years ago by mkommend
comment:3 Changed 8 years ago by mkommend
- Owner changed from mkommend to abeham
- Status changed from accepted to reviewing
r14793: Improved performance of RunCollectionView and added runs counter.
The performance has been improved by the following changes:
- Used a fixed image list containing only two images (run and empty) and reused those images. As a result I could remove the UpdateListViewItemImage method and the RunCollectionView does not react to image changes anymore. However, this has never been support for runs.
- Accessed run.Name directly instead of run.ToString() in listview item creation
- Used a fixed tool tip instead of accessing the ItemAttribute for every run
- Used AddRange and Clear methods of itemlistView when adding or resetting the run collection. Removed AddListViewItem method.
Addditional thought: Item.ItemImageChanged is only support for optimizers (execution state overlay). However, every view has to register to this event and act accordingly. Maybe an interface IImageItem with the event would be appropriate.
comment:4 Changed 7 years ago by abeham
- Owner changed from abeham to mkommend
- Status changed from reviewing to readytorelease
Reviewed r14793 and tested opening attached file: ok, no further comments.
comment:5 Changed 7 years ago by mkommend
- Resolution set to done
- Status changed from readytorelease to closed
Maybe related to #1719 that talks about using virtual mode for the RunCollectionView.