You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Gilles Sadowski (Jira)" <ji...@apache.org> on 2020/10/08 14:51:00 UTC
[jira] [Resolved] (MATH-1555) Wrong declaration of ATan2 class
[ https://issues.apache.org/jira/browse/MATH-1555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gilles Sadowski resolved MATH-1555.
-----------------------------------
Fix Version/s: 4.0
Resolution: Fixed
Fixed in commit 27f289f6be1396893247f13225a52388f598a8d7 ("master" branch).
Thanks for the report.
> Wrong declaration of ATan2 class
> --------------------------------
>
> Key: MATH-1555
> URL: https://issues.apache.org/jira/browse/MATH-1555
> Project: Commons Math
> Issue Type: Bug
> Reporter: Laurent Galluccio
> Priority: Major
> Fix For: 4.0
>
> Original Estimate: 1m
> Remaining Estimate: 1m
>
> If i check the content of ATan2 class, the value method is coded like this
> public double value(double x, double y) {
> return FastMath.atan2(x, y);
> }
> with
> *x* Abscissa for which the function value should be computed.
> *y* Ordinate for which the function value should be computed.
>
> But the FastMath.atan2 method is declared like this
> /**
> * Two arguments arctangent function
> * @param y ordinate
> * @param x abscissa
> * @return phase angle of point (x,y) between \{@code -PI} and \{@code PI}
> */
> public static double atan2(double y, double x)
>
> Which means that ordinate and abscissa are inverted. This should be fixed.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)