-
Notifications
You must be signed in to change notification settings - Fork 35
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
[WIP] Add torchgeo models #233
Conversation
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
By running the automatic tests |
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
@paolo-fraccaro Should we replace the current torchgeo version from requirements and pyproject.toml by: |
@Joao-L-S-Almeida I think it makes sense to get the latest. Or is there a way to point to a specific commit? |
@paolo-fraccaro It's possible to add the hash at the end: |
OK that may be the best solution. |
Please also check if there is any other library that requires the same approach. |
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
@paolo-fraccaro I tried to use the latest torchgeo but: |
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @paolofraccaro cc @Joao-L-S-Almeida
- merged
Added different models as implemented in torchgeo. Added a related test file and passing all other tests. Only issue is mmseg installation imcompatible with updated version of timm.