On 02/15/2012 02:41 PM, Patrick Meyer wrote: > OK, I submited a patch that includes comments and documentation. Let me know > if I need to write more, but I think I've covered the functionality of the > classes.
Hi Patrick, thanks for the patch. I have applied it together with additional code cleanup and javadoc in r1244667. What I have done: - applied your javadoc patch - added missing javadoc for variables - moved var initialization to constructor - changed exceptions to more specific ones - simplified getCovarianceMatrix (uses getData() to construct the RealMatrix) - changed deltaX / deltaY to local variables Could you please take a look and give feedback about the changes made? @all: from my point of view this would resolve issue MATH-449 so far as it is fully documented and has proper unit tests. Thanks Thomas --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
