Uploaded image for project: 'Commons Math'
  1. Commons Math
  2. MATH-626

Move RealVector.sparseIterator() to SparseRealVector

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: 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.

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: