Commons Math
  1. Commons Math
  2. MATH-626

Move RealVector.sparseIterator() to SparseRealVector

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Having this method available for all is not very efficient, because if I want use this feature I must check for interface SparseRealVector.
      otherwise different implementations could be faster.
      I.e. no array clone and the assigning all values again.
      Maybe one could add a menthod fillstate() to SparseRealVector to guide which is faster - use array clone and then sparseIterator or just create the array and then fill it up.

      To reduce confusion a second Interface ArrayRealVector with the method getDataRef() and removing getData() from RealVector would be nice - there is a toArray() this should be sufficient.

        Issue Links

          Activity

          Arne Plöse created issue -
          Phil Steitz made changes -
          Field Original Value New Value
          Fix Version/s 3.1 [ 12317576 ]
          Affects Version/s 3.0 [ 12314824 ]
          Gilles made changes -
          Fix Version/s 4.0 [ 12317577 ]
          Fix Version/s 3.1 [ 12317576 ]
          Gilles made changes -
          Link This issue is part of MATH-765 [ MATH-765 ]
          Thomas Neidhart made changes -
          Link This issue is superceded by MATH-870 [ MATH-870 ]
          Thomas Neidhart made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 4.0 [ 12317577 ]
          Resolution Won't Fix [ 2 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Arne Plöse
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development