[
https://issues.apache.org/jira/browse/MATH-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13068453#comment-13068453
]
Gilles commented on MATH-623:
-----------------------------
If nobody objects, and since one less {{clone}} is good, I'll change the
{{add(double[] v)}} method as per Arne's suggestion.
> Use implementation of JAMA for ArrayRealVector operators
> --------------------------------------------------------
>
> Key: MATH-623
> URL: https://issues.apache.org/jira/browse/MATH-623
> Project: Commons Math
> Issue Type: Improvement
> Affects Versions: 3.0
> Reporter: Arne Plöse
> Priority: Minor
> Attachments: ArrayRealVector.diff
>
>
> For instance the add(double[] v) fist clones the array, and then adds all
> entries of this.data to the result.
> JAMA uses the following approach create a empty result[] the assign the sum
> of each entry to the result. this is approximately 10 -20 % faster.
> I will provide a patch with some more enhancements/cleanups for this.
> in the test look for XXX:
> the first number is the jama time the second the current algorithm.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
