lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Muir (JIRA)" <>
Subject [jira] [Updated] (LUCENE-7125) remove BKD_TOLERANCE from LatLonPoint polygon tests
Date Mon, 21 Mar 2016 23:51:25 GMT


Robert Muir updated LUCENE-7125:
    Attachment: LUCENE-7125.patch

Here is a patch. I think we can improve the test more. The base test class i think it should
sometimes explicitly quantize the random polygons to improve the possibility we land exactly
on edge cases.

> remove BKD_TOLERANCE from LatLonPoint polygon tests
> ---------------------------------------------------
>                 Key: LUCENE-7125
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>         Attachments: LUCENE-7125.patch
> Currently this rejects any rectangles within a delta as being "too close to an edge",
and returns null, which means those are not tested in the random test.
> But we already compensate for quantization in the test, we should just remove this additional
> The only problem I found in beasting is that, due to the algorithm being used for polygons,
it obeys the "definition of insideness" described here:
> In my opinion, this test should work just like the distance test, and just ensure the
query is consistent with {{GeoRelationUtils.pointInPolygon()}}. Anything the query is doing
other than running that in brute-force is purely an optimization. This also makes it easier
to test more interesting polygons in the future other than just rectangles.
> Separately, we can test that this basic method works correctly better if we want, but
that can just be a unit test.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message