Project

General

Profile

Actions

Backport #2261

closed

lib/matrix: following the extensive changes

Added by marcandre (Marc-Andre Lafortune) over 14 years ago. Updated almost 13 years ago.


Description

=begin
(a) Shouldn't r25412 (and other related commits) be also applied to the 1.8 branch?

(b) r25412 introduced Matrix#inspect_org. It is not documented.

Its output relies on the way the matrix information is stored internally by the Matrix class. Shouldn't this be considered implementation details and thus subject to change and not for public consumption?

It should either:

  • be removed,
  • be made private (although it is not used by the implementation) or
  • be documented

I recommend the first option

(c) #compare_by_row_vectors and #compare_by should either:

  • be removed (they are not used by the implementation anymore),
  • be made private or
  • be documented.

I recommend the first option
=end

Actions

Also available in: Atom PDF

Like0
Like0Like0