Opened 6 weeks ago

Last modified 3 weeks ago

#3100 reviewing defect

TriangularMatrix uses wrong base class

Reported by: bburlacu Owned by: jkarder
Priority: medium Milestone: HeuristicLab 3.3.17
Component: Data Version: trunk
Keywords: Cc:

Description

Since the underlying storage is an array, the TriangularMatrix<T> class should inherit from ValueTypeArray<T> instead of ValueTypeMatrix<T>. This would lead to a more efficient implementation.

Optional enhancements:

  • option to specify storage order (row- or column major)
  • a property to access the underlying T[] array

Change History (3)

comment:1 Changed 6 weeks ago by bburlacu

  • Status changed from new to accepted

comment:2 Changed 5 weeks ago by bburlacu

  • Milestone set to HeuristicLab 3.3.17
  • Owner changed from bburlacu to architects
  • Status changed from accepted to reviewing
  • Version set to trunk

r17831: Use ValueTypeArray as base class for TriangularMatrix.

  • Data property exposes the underlying array as a ReadOnlyCollection.
  • No need to specify storage order since the matrix is symmetric.

comment:3 Changed 3 weeks ago by bburlacu

  • Owner changed from architects to jkarder
Note: See TracTickets for help on using tickets.