Adam J. Stewart

Results 884 comments of Adam J. Stewart

This approach doesn't work well for datasets like L7 Irish or L8 Biome where a single `RasterDataset` is used for both image and mask. I think the solution is to...

You can play around with other enums and see which looks best: https://rasterio.readthedocs.io/en/stable/api/rasterio.enums.html#rasterio.enums.Resampling I agree with @DimitrisMantas that we should pick a safe/simple/fast default.

@ashnair1 is this waiting on kornia 0.7.4 for something?

> Will need to push a fix to kornia to allow non-float dtypes. Any progress on this?

We recently updated the repo to use Python 3.10+. Some of your code uses Python 3.9 syntax. You can fix this with pyupgrade: https://torchgeo.readthedocs.io/en/stable/user/contributing.html#linters

@ianturton is this still a work in progress? We're preparing for a new release in the next couple weeks. Trying to merge or close as many remaining PRs as possible.

Related to #1830 and #1887. We spoke on Slack, I think you should just take over #1887 and finish it up. We would love to get this into the 0.5.2...

I've removed codecov as a required status check since it almost never passes. For now, we'll have to manually verify whether or not coverage really decreases. There is some movement...

Seems like SMP is now maintained, we just need to start contributing PRs there instead of to TorchSeg.

I'm unable to reproduce this issue. I tried both 2017 and 2022 and both downloaded fine on my system. What version of torchvision are you using? Can you try upgrading...