Opened 7 years ago

Closed 6 years ago

#2018 closed enhancement (obsolete)

Pasting values in a StringConvertibleMatrix is rather slow

Reported by: mkommend Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.x Backlog
Component: Data.Views Version: branch
Keywords: Cc:

Description (last modified by sforsten)

This is especially an issue if more than 10,000 values are updated. My suspicion is that due to missing batch update methods every value must be set individually (which fires an event, updates the view, etc.).

StringConvertibleArray also has to be improved.

Change History (11)

comment:1 Changed 7 years ago by mkommend

  • Description modified (diff)
  • Owner changed from swagner to sforsten
  • Status changed from new to assigned

comment:2 Changed 7 years ago by sforsten

  • Status changed from assigned to accepted

r9273: create new branch
r9274: branch project Data
r9275: branch project Data.Views
r9278: branch project Problems.DataAnalysis
r9279: branch project Optimization
r9280: branch project Analysis
r9281: branch project Problems.TravelingSalesman
r9282: branch project Problems.TestFunctions
r9283: branch project Problems.VehicleRouting
r9284: branch project Problems.QuadraticAssignment
r9285: branch project Problems.ArtificialAnt

r9286:

  • added new methods to the interface IStringConvertibleMatrix as well as two structs. Also the event ItemChanged has been changed to ItemsChanged
  • class ValueTypeMatrix now implements IStringConvertibleMatrix instead of the classes which inherit from it
  • small changes have been applied to a lot of classes to correctly implement the changed interface IStringConvertibleMatrix
  • solution file, Build.cmd and PreBuildEvent.cmd have been added

Updating about 10,000 values now takes far below 1 second.

comment:3 Changed 7 years ago by sforsten

r9306:

  • renamed the structs and methods in IStringConvertibleMatrix
  • added MatrixValuesChangedEventArgs in IStringConvertibleMatrix
  • added methods SetValues(MatrixValues<T>) in ValueTypeMatrix
  • fixed bugs in StringConvertibleMatrixView: DataGridView has now at least one column and dataGridView_CellValidating does not set e.Cancel to true anymore.

comment:4 Changed 7 years ago by sforsten

r9307: restored original behaviour of StringConvertibleMatrixView

comment:5 Changed 7 years ago by sforsten

  • Description modified (diff)
  • Owner changed from sforsten to mkommend
  • Status changed from accepted to reviewing

r9308:

  • added column names to the ValueTypeArray and StringArray
  • added batch update methods to IStringConvertibleArray similar to methods in IStringConvertibleMatrix

comment:6 Changed 7 years ago by sforsten

  • Version changed from 3.3.7 to branch

comment:7 Changed 6 years ago by sforsten

r9401: renamed column to element in array types

comment:8 Changed 6 years ago by mkommend

  • Status changed from reviewing to assigned

r9422: Updated StringConvertibleMatrixView branch with the latest trunk changes.

comment:9 Changed 6 years ago by mkommend

  • Status changed from assigned to accepted

comment:10 Changed 6 years ago by mkommend

Pasting values was improved with #2042.

comment:11 Changed 6 years ago by sforsten

  • Resolution set to obsolete
  • Status changed from accepted to closed

The problem was the ToString method of ValueTypeMatrix/-Array and StringConvertibleMatrix /-Array. This has been fixed in #2042. Therefore, this ticket is obsolete.

r9446: deleted branch ImprovingStringConvertibleMatrix

Note: See TracTickets for help on using tickets.