[
https://issues.apache.org/jira/browse/LUCENE-8136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16338522#comment-16338522
]
Karl Wright commented on LUCENE-8136:
-------------------------------------
The problem is that MINIMUM_ANGULAR_RESOLUTION is tied precisely to
MINIMUM_RESOLUTION. GeoPolygonFactory uses isIdentical() to see if points are
the same, for instance, and that uses MINIMUM_RESOLUTION.
MINIMUM_ANGULAR_RESOLUTION is used only when comparing angles. So a change to
one must be accompanied by a change to the other.
I verified that MINIMUM_RESOLUTION of 1e-8 is needed to make the plane issue go
away. That's four orders of magnitude greater than our current value, and
clearly too coarse.
I still think there must be an alternative calculation that could be used given
that the resulting plane can readily be checked for accuracy against the
original points. Maybe an iterative adjustment might work. Will try that next.
> Planes constructed with two points are wrong when points are close
> ------------------------------------------------------------------
>
> Key: LUCENE-8136
> URL: https://issues.apache.org/jira/browse/LUCENE-8136
> Project: Lucene - Core
> Issue Type: Bug
> Components: modules/spatial3d
> Reporter: Ignacio Vera
> Assignee: Karl Wright
> Priority: Major
> Attachments: LUCENE-8136-RandomTest.patch,
> LUCENE-8136-RandomTest.patch
>
>
> Whenever a plane is constructed with two points (and the center of the
> planet), and those points are close to each other,it might happen that the
> final plane does not contain all points used for the construction. The issue
> seems to happen when the resulting magnitude of the normal vector of the new
> plane (calculated using cross product) is lower that 1e-5.
> This is a follow up of issue LUCENE-8133.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
