[ https://issues.apache.org/jira/browse/MATH-688?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13123996#comment-13123996
]
Gilles commented on MATH-688:
-----------------------------
OK, I now understand that there is no problem making the change then.
> "FastMath" contains fractions of constants
> ------------------------------------------
>
> Key: MATH-688
> URL: https://issues.apache.org/jira/browse/MATH-688
> Project: Commons Math
> Issue Type: Improvement
> Reporter: Gilles
> Assignee: Gilles
> Priority: Trivial
> Fix For: 3.0
>
>
> Fractions such as 1.0 / 3.0, 1.0 / 5.0, ... could be replaced with static final constants.
> In "asinh", for example, the change can sometimes result in more than halving the computation
time (when the argument is in the range of values where the constants are used).
> In absolute times, the gain is not huge, amounting (on my machine) to something like
370 ms for 10 million computations.
> And, of course, the downside will be that these constants will be computed at the loading
of "FastMath", leading to an additional penalty of about 16 divisions at initialization...
:)
--
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
|