Blake Thompson

Results 29 issues of Blake Thompson

It is not intuitive right away how the intersection processing and the snap rounding is done in the code. Documentation should be added and put in the `/docs` directory of...

Added a new test after extensive fuzzing that found a new failure case. The testcase has been added in the https://github.com/mapbox/wagyu/tree/spike_failure branch. Resulting polygon with failure: ![image](https://user-images.githubusercontent.com/1794907/71740211-3cf97000-2e21-11ea-85db-7b4dbbfab17e.png) Original polygon from...

It would be very interesting to see how Wagyu compares against GEOS/MakeValid in PostGIS both from performance and accuracy. It seems that the method for PostGIS is located [here](https://github.com/postgis/postgis/blob/bd3177743db5cb786deb05d7327f39a5cec2daf6/liblwgeom/lwgeom_geos_clean.c#L373-L667) and...

For features that are completely inside a bounding box, it might be faster to simply copy the entire feature at once rather then running through the quick clipping algorithm. However,...

Currently the wagyu algorithm only works with integers, however, with some fine tuning it might be possible to support floating point numbers. ## Steps: - [ ] Use Approximate equality...

Currently we only generate a single ring of random points, perhaps this doesn't cover all the possible edge cases. It might be a good choice to change this to multiple...

The testcases seem to be filled with multipolygon test case failures on validity, but I do am not certain there is in fact any validity errors. I think this may...

We had the start of some work towards using https://github.com/mapnik/geometry-test-data/ for fixture testing as part of Wagyu and this is excellent! However, it is very difficult to see differences in...

This is the master ticket to discuss what needs to be done prior to releasing version 1.0 of wagyu. Updates to follow

`sanitize-caja` is now deprecated and replaced by `@mapbox/sanitize-caja`. This dependency should be updated. Additionally, we should consider moving this package from `mapbox.js` to `@mapbox/mapbox.js`.