first draft of array clamper

mapbox / nodata

Because the pixels you can't see are harder than the ones you can.

MIT License

1 stars 3 forks source link

first draft of array clamper #52

Closed dnomadb closed 6 years ago

dnomadb commented 8 years ago

This is possibly a small speedup that integrates into existing functionality.

Instead of sending the entire "has no data" window array over to rasterio.fill.fillnodata, it "clamps" down to the non-nodata area + pads out to the filling area as to minimize the amount of data copied.

To do:

[ ] Add more explicit unit tests
[ ] Verify that buffering amount is enough (should it be max fill dist + 1?)

cc @perrygeo

dnomadb commented 8 years ago

Obvi. smthing is effed; investigating.

perrygeo commented 8 years ago

dnomadb commented 8 years ago

I wonder how much this logic could integrate with what we are doing in skippy?

dnomadb commented 6 years ago

This is no longer necessary, closing.