commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sébastien Brisard (JIRA) <j...@apache.org>
Subject [jira] [Commented] (MATH-659) Remove solve(double[][]) from DecompositionSolver
Date Thu, 08 Sep 2011 06:05:09 GMT

    [ https://issues.apache.org/jira/browse/MATH-659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13100096#comment-13100096
] 

Sébastien Brisard commented on MATH-659:
----------------------------------------

In rev1166533, I removed the call to this method in {{AdamsNordsieckTransformer.initializeHighOrderDerivatives(double,
double[], double[][], double[][])}}. This raises one important question. All across the package
o.a.c.m.ode, matrix variables are explicitely declared as {{Array2DRowRealMatrix}} (as opposed
to {{RealMatrix}}). This leads to an ugly cast (see the end of the presently modified method).
I was wondering how necessary that was. Would it be thinkable to use the most general supertype
for all matrices (?)

> Remove solve(double[][]) from DecompositionSolver
> -------------------------------------------------
>
>                 Key: MATH-659
>                 URL: https://issues.apache.org/jira/browse/MATH-659
>             Project: Commons Math
>          Issue Type: Task
>    Affects Versions: 3.0
>            Reporter: Sébastien Brisard
>            Assignee: Sébastien Brisard
>            Priority: Trivial
>              Labels: linear
>
> Following MATH-653, where {{double[]}} were removed from {{RealVector}}, and {{double[]
solve(double[])}} was removed from {{DecompositionSolver}}, the method {{double[][] solve(double[][])}}
should be removed from {{DecompositionSolver}}. {{RealMatrix solve(RealMatrix)}} should be
called instead.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message