Opened 7 months ago

Closed 4 months 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)

RunCollection Test 10K.hl (798.3 KB) - added by mkommend 7 months ago.

Download all attachments as: .zip

Change History (6)

Changed 7 months ago by mkommend

comment:1 Changed 7 months ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 7 months ago by mkommend

Maybe related to #1719 that talks about using virtual mode for the RunCollectionView.

comment:3 Changed 7 months 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 4 months 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 4 months ago by mkommend

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

r15115: Merged r14793 into stable.

Note: See TracTickets for help on using tickets.