-
Notifications
You must be signed in to change notification settings - Fork 200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI failing after cargo update
#1260
Comments
Notably, tests pass if I pin to the previous rstar - rstar = "0.12.0"
+ rstar = "=0.12.0" (latest version is 0.12.1) Still investigating... |
Does testbuilder have a concave hull function? |
Git bisect tells me this is the commit that introduced the failure: I'm not trying to imply there's anything wrong with that commit - it could just as likely have surfaced an existing bug in geo (or at least in our test). |
My understanding is that concave hull does not have a rigorous definition, so comparing to an existing implementation would be directionally helpful, but not useful for a unit test like this, unless our implementation was based on JTS's (which it was not AFAIK). |
BTW if you don't have JTS test builder, you should download it now. I use it all the time! You can download from their GH release page: https://github.com/locationtech/jts/releases/tag/1.20.0 Launch with |
Hm that concave hull function has always been kinda troublesome, including in the very recent past. |
Oh I do just not at the big screen rn (but yes everyone should have it!) |
I think the root cause is a bug in rstar - see georust/rstar#181 |
Fixed in rstar, released yesterday. |
https://github.com/georust/geo/actions/runs/11689401597/job/32552055508
Failure is in concave hull.
The text was updated successfully, but these errors were encountered: