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

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          820d 13h 16m 1 Thomas Neidhart 18/Oct/13 23:49
          Thomas Neidhart made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 4.0 [ 12317577 ]
          Resolution Won't Fix [ 2 ]
          Thomas Neidhart made changes -
          Link This issue is superceded by MATH-870 [ MATH-870 ]
          Hide
          Thomas Neidhart added a comment -

          Obsolete after resolution of MATH-870.

          Show
          Thomas Neidhart added a comment - Obsolete after resolution of MATH-870 .
          Hide
          Sébastien Brisard added a comment -

          See also this recent discussion.

          Show
          Sébastien Brisard added a comment - See also this recent discussion .
          Gilles made changes -
          Link This issue is part of MATH-765 [ MATH-765 ]
          Gilles made changes -
          Fix Version/s 4.0 [ 12317577 ]
          Fix Version/s 3.1 [ 12317576 ]
          Hide
          Gilles added a comment -

          Wouldn't it be an incompatible change? I'm referring to the title of this issue, not the various other suggestions that should have been in their own JIRA requests.

          Show
          Gilles added a comment - Wouldn't it be an incompatible change? I'm referring to the title of this issue, not the various other suggestions that should have been in their own JIRA requests.
          Phil Steitz made changes -
          Field Original Value New Value
          Fix Version/s 3.1 [ 12317576 ]
          Affects Version/s 3.0 [ 12314824 ]
          Hide
          Phil Steitz added a comment -

          If we do implement this, it can wait until 3.1

          Show
          Phil Steitz added a comment - If we do implement this, it can wait until 3.1
          Arne Plöse created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development