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 (JIRA)" <ji...@apache.org> on 2018/09/24 00:26:00 UTC

[jira] [Commented] (GEOMETRY-21) Investigate Norm Accuracy

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

Gilles commented on GEOMETRY-21:
--------------------------------

{quote}failure
{quote}
Inserting some "print" statements:
{noformat}
0.10617094689696915 > 0.10617094689696706
0.10617094689696915 > 0.10617094689696706
failAssertCount=2 totalAssertCount=387
{noformat}
Thus, 2 failures (where the expectation is on the reversed inequality) that would pass if tolerance >= 1e-15 for a total of 387 similar inequality tests that pass.

{{testGeographicalMap()}} is a fairly big unit test, and letting the code continue past the above check completes successfully.

Commit be34ad93c0b0554ce5927811e0f762312172b9ea makes the test pass.
 Please have a look.

Also: magic numbers should be avoided (constant must be declared as a {{static final}} variable).

> Investigate Norm Accuracy
> -------------------------
>
>                 Key: GEOMETRY-21
>                 URL: https://issues.apache.org/jira/browse/GEOMETRY-21
>             Project: Apache Commons Geometry
>          Issue Type: Task
>            Reporter: Matt Juntunen
>            Priority: Minor
>
> Based on discussion in GEOMETRY-17, we should investigate the floating point accuracy of the current Vector normalization methods. Specifically, when the UnitVector private subclass in Vector3D is implemented to return exactly 1.0, the SphericalPolygonsSetTest#testGeographicalMap unit test in commons-geometry-enclosing begins to fail. We should
>  # Determine the cause of this failure.
>  # Determine if the current approach with the UnitVector subclasses introduces any issues with floating point accuracy.
>  # Add unit tests for Vector[123]D to quantify and verify the accuracy of the normalization.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)