Adam J. Stewart
Adam J. Stewart
Went back to the original PR implementation to solve the Sphinx warning.
@sgillies is this PR and #3352 #3351 still wanted? If not I'll close, trying to clean out old PRs.
The comments seem to suggest that this faces many of the same problems that this PR is designed to solve, but I'm happy to try it and see.
@snowman2 @sgillies ready to merge?
I'm also thinking about changing the variant defaults from the minimal build to the default build like we did with PyTorch. This will enable numa by default. But want to...
Ping @fgava90
Note that if I switch to `boundless=False` on the line @calebrob6 mentioned, almost all of TorchGeo's tests pass. However, I'm not confident we are properly testing out-of-bounds reads. Let me...
Only other change that bit us was that `rasterio.merge.merge` seems to be much more picky about transform orientation: ``` ValueError: Bounds must both have the same orientation ```