Free cookie consent management tool by TermsFeed Policy Generator

Opened 7 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)

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

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by mkommend

comment:1 Changed 7 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 7 years ago by mkommend

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

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

r15115: Merged r14793 into stable.

Note: See TracTickets for help on using tickets.