Skip to content
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

Support additional SatlasPretrain models. #1861

Open
1 of 4 tasks
piperwolters opened this issue Feb 6, 2024 · 6 comments
Open
1 of 4 tasks

Support additional SatlasPretrain models. #1861

piperwolters opened this issue Feb 6, 2024 · 6 comments
Labels
models Models and pretrained weights
Milestone

Comments

@piperwolters
Copy link
Contributor

piperwolters commented Feb 6, 2024

Checkpoint

  • Single-Image Swin Backbones
  • Multi-Image Swin Backbones
  • Other Backbones (Swin-Tiny, Resnets)
  • Decoders

Summary

I see that a couple SatlasPretrain models have been merged and will be included in the 0.6.0 release, which is great!
I'm hoping that additional pretrained weights and models could be integrated, specifically the Sentinel-1 and Landsat models as well as models with varying number of input images and bands.

We recently released the satlaspretrain-models package which makes it easy to load the various pretrained models. I could make a PR to torchgeo to include more pretrained models, either 1) using the code that was put in place in the above PR or 2) using this new satlaspretrain-models package. Wondering if you all have a preference?
Thanks!

Rationale

SatlasPretrain models improve performance and speed up training on downstream tasks, as opposed to training from scratch. It would be great to have these models easily accessible for torchgeo users.

Implementation

Either follow the pattern from this merged PR or utilize the satlaspretrain-models package.

Alternatives

No response

Additional information

No response

@adamjstewart adamjstewart added the models Models and pretrained weights label Feb 7, 2024
@adamjstewart
Copy link
Collaborator

Yes, we would love to get the rest of the SatlasPretrain models in TorchGeo so they're as easy as possible for our users to access!

My preference would be for option 1. This avoids adding "yet another dependency" and integrates nicely with our Lightning trainers. Note that we've really only added backbones in TorchGeo so far. We would like to add decoders as well, but haven't yet put a lot of thought into how the API should work.

P.S. When we added the first couple SatlasPretrain models, we assumed they were under the Apache-2.0 license, but based on this new repo it seems they are actually ODC-BY. Is this correct? I'll update our documentation if so.

FYI @favyen2

@adamjstewart adamjstewart added this to the 0.6.0 milestone Feb 7, 2024
@piperwolters
Copy link
Contributor Author

Okay, sounds good. I will just start by adding the additional backbones in a PR, and if you all come up with a plan or skeleton for decoders, I can make a PR for that later.

And yes, our data and model checkpoints are under ODC-BY and the code is Apache 2.0.
Thank you!

@isaaccorley
Copy link
Collaborator

@piperwolters I added the previous Satlas checkpoints. If you need any help let me know! We are also active on the TorchGeo slack channel if you want to discuss.

@adamjstewart
Copy link
Collaborator

Is this completed by #1884?

@piperwolters
Copy link
Contributor Author

@adamjstewart That first PR only added single-image Swin backbones. Would like to add multi-image backbones and decoders eventually, but don't necessarily need to keep this issue open for that.

@adamjstewart
Copy link
Collaborator

Nah, let's keep it open. You could add a bullet point list to the issue like:

  • Swin Sentinel single-image
  • Swin Sentinel multi-image

if you want to track progress on completion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
models Models and pretrained weights
Projects
None yet
Development

No branches or pull requests

3 participants