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

Karl Wright commented on LUCENE-6759:
-------------------------------------

Hi Mike,

This code demonstrates the arrangement, as far as I have been able to research 
so far:

{code}
    c = new 
GeoCircle(PlanetModel.SPHERE,0.004729080495762766,-7.023094095867159E-5,0.002549346432178953);
    p1 = new 
GeoPoint(PlanetModel.SPHERE,0.007197150578698408,0.0010389455251423893);
    assertTrue(!c.isWithin(p1));
    xyzb = new XYZBounds();
    c.getBounds(xyzb);
    area = GeoAreaFactory.makeGeoArea(PlanetModel.SPHERE, 
      xyzb.getMinimumX(), xyzb.getMaximumX(), xyzb.getMinimumY(), 
xyzb.getMaximumY(), xyzb.getMinimumZ(), xyzb.getMaximumZ());
    relationship = area.getRelationship(c);
    assertTrue(relationship == GeoArea.OVERLAPS || relationship == 
GeoArea.WITHIN);
    // Point is actually inside the bounds, but outside the shape
    assertTrue(area.isWithin(p1));
{code}

So somewhere along the line during the bkd descent for the point in question, 
the point is being included when it shouldn't be, possibly because a CONTAINS 
result is being returned for a subarea.  But we don't yet know what that is.

[~mikemccand], what I think we need for debugging situations of this kind is a 
test method that can be called that starts with a shape and descends solely 
with the view of analyzing a single provided point.  It may be that your 
architecture works great for this already, but I'm not sure.  For the moment 
I'm going to try assuming that the problem is a CONTAINS result that should 
have been an OVERLAPS, and adjusting the MINIMUM_RESOLUTION etc.



> Integrate lat/long BKD and spatial 3d, part 2
> ---------------------------------------------
>
>                 Key: LUCENE-6759
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6759
>             Project: Lucene - Core
>          Issue Type: New Feature
>            Reporter: Michael McCandless
>         Attachments: LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch, 
> LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch
>
>
> This is just a continuation of LUCENE-6699, which became too big.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to