Enable Rtree for all bounding boxes#686
Conversation
mschoema
left a comment
There was a problem hiding this comment.
I'm wondering if we should keep this for 1.4. The implementation is in my opinion not complete, and we shouldn't add it half-baked to 1.3.
|
Perfect to leave it for 1.4. I will correct the issues in a forthcoming PR. |
IMHO the last commit solved all issues. The changes of this PR basically does not change the algorithm only enable bounding boxes of varying sizes. For me it still can go in 1.3. |
Fair, the RTrees are already there anyway. I still think there are things left to do, but this PR can already be added to 1.3-alpha1. Maybe I'll find the time to improve the RTrees further before the actual 1.3 release. |
Thanks ! This enables to be used for other bindings, in particular DuckDB. |
No description provided.