Commons Math
  1. Commons Math
  2. MATH-875

Deprecate RealVector.sparseIterator()

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 3.3
    • Labels:

      Description

      Following MATH-870, RealVector.sparseIterator() has become useless. Its interface was confusing. Indeed, the Javadoc states

      Specialized implementations may choose to not iterate over all dimensions, either because those values are unset, or are equal to defaultValue(), or are small enough to be ignored for the purposes of iteration.

      The reference to defaultValue() (which does not exist) suggests that the interface offers the flexibility to chose the value which is not stored in the sparse implementation. On the other hand, "small enough to be ignored" suggests that this default value is zero.

      In version 3.1, this method will be deprecated, to be removed in 4.0.

        Activity

        Hide
        Sébastien Brisard added a comment -

        In r1395455, deprecated sparseIterator() in the RealVector hierarchy, and replaced its use with iterator().

        Show
        Sébastien Brisard added a comment - In r1395455 , deprecated sparseIterator() in the RealVector hierarchy, and replaced its use with iterator() .
        Hide
        Sébastien Brisard added a comment -

        This improvement no longer affects version 3.1. As of version 4.0, the method must be removed.

        Show
        Sébastien Brisard added a comment - This improvement no longer affects version 3.1. As of version 4.0, the method must be removed.
        Hide
        Luc Maisonobe added a comment -

        Un-deprecated method in r1570510.
        The documentation explicitly states that ignored entires are the exact zero ones.

        This is part of Apache Commons Math reconsidering support for sparse
        linear algebra.

        Show
        Luc Maisonobe added a comment - Un-deprecated method in r1570510. The documentation explicitly states that ignored entires are the exact zero ones. This is part of Apache Commons Math reconsidering support for sparse linear algebra.
        Hide
        Luc Maisonobe added a comment -

        Closing all resolved issue now available in released 3.3 version.

        Show
        Luc Maisonobe added a comment - Closing all resolved issue now available in released 3.3 version.

          People

          • Assignee:
            Unassigned
            Reporter:
            Sébastien Brisard
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development