iverase commented on issue #13841: URL: https://github.com/apache/lucene/issues/13841#issuecomment-2446320111
>According to [GeoJSONLint](https://geojsonlint.com/) this polygon doesn't follow the "right-hand rule". I don't think that matters as the tessellation algorithm would order the rings before any processing. I had a look and the problem is in the following check which is not correct: https://github.com/apache/lucene/blob/9359cfd32f1f2c9ec1fd7e3cabd75ada4fc91204/lucene/core/src/java/org/apache/lucene/geo/Tessellator.java#L463 This check as added to support shared edges by multiple holes like this example: <img width="536" alt="image" src="https://github.com/user-attachments/assets/0a065f09-4132-4409-9be0-42b7a1e33f50";> We need to improve that area to fix this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
