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

David Smiley commented on LUCENE-8139:
--------------------------------------

Ignacio, you've had the authorizations to commit to Lucene/Solr since January 
11th (Adrien told you then).  If you have any questions, let us know :-).  In 
general, "ant precommit" and passing tests is the biggest thing.  And I think 
you've observed the JIRA workflow.

> polygon constructor can't reliably find a point inside the polygon
> ------------------------------------------------------------------
>
>                 Key: LUCENE-8139
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8139
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/spatial3d
>            Reporter: Ignacio Vera
>            Assignee: Karl Wright
>            Priority: Minor
>             Fix For: 6.7, master (8.0), 7.3
>
>         Attachments: LUCENE-8139.patch
>
>
> The polygon factory uses a randomize algorithm to find a point inside of the 
> polygon. This might fail, in particular when polygons are very small. I want 
> to propose a small improvement; Before going into the randomize algorithm, we 
> can first try to try a point defined by the center of mass of the provided 
> points. This approach improves things, for example for very small polygons 
> with three points.
>  
> does it make sense?



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to