[
https://issues.apache.org/jira/browse/MATH-746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13221359#comment-13221359
]
Gilles commented on MATH-746:
-----------------------------
bq. FastMath still needs a bit of tidying up [...]
I've reinstated the "final" qualifier for "RECOMPUTE_TABLES_AT_RUNTIME". But I
don't understand the remark on "FastMathCalc". Strictly speaking, it is not
needed, but it is the "link" to the meaning of the tables contents. It's not
necessary to use "FastMath", but it is in order to understand how the tables
are built.
What could have been done was to have "FastMathCalc" be a class similar to
"FastMathLiteralArrays", in the sense that it would contain all the methods
"loadExpIntA", ... but each method would, instead of returning a precomputed
array, trigger the necessary computations and populate the array just before
returning it.
What could have been moved away (to the "test" part of the source tree) are the
support methods that print the arrays.
Both can still be done after releasing 3.0.
> Things to do before releasing 3.0
> ---------------------------------
>
> Key: MATH-746
> URL: https://issues.apache.org/jira/browse/MATH-746
> Project: Commons Math
> Issue Type: Task
> Reporter: Gilles
> Priority: Blocker
> Fix For: 3.0
>
>
> This issue is meant to contain a list of tasks to be completed before the
> release.
> * Remarks to be added to the *release notes*:
> ** Experimental code: {{BOBYQAOptimizer}} (cf. MATH-621) (/) (r1291896)
> *** Many code paths untested
> *** Looking for volunteers to improve the code readability, robustness and
> performance
> *** Looking for volunteers to extend the test suite
> ** {{FastMath}} is not always faster than {{Math}} (issue MATH-740) (/)
> (r1291896)
> ** List of new features (/) (r1292318)
> * Create a "release branch"
> * Disable CheckStyle scanning of {{BOBYQAOptimizer}} (/) (r1244855)
> * Comment out "print" statements in {{BOBYQAOptimizerTest}} (/) (r1244975)
> * Remove unit test class {{BatteryNISTTest}}
> * Remove class {{PivotingQRDecomposition}}
> * Comment out "print" statement in {{SymmLQTest}} (/) (r1244992, removed in
> r1244996)
> * Findbugs:
> ** org.apache.commons.math3.analysis.solvers.BaseSecantSolver (/) (r1292245)
> ** org.apache.commons.math3.distribution.TriangularDistribution (/) (r1292572)
> ** org.apache.commons.math3.optimization.direct.BOBYQAOptimizer (cannot be
> corrected before release)
> ** org.apache.commons.math3.stat.regression.RegressionResults (/) (r1292251
> and r1292270)
> * Check the user guide
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
