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

Change "classification_model" to "model" #916

Merged
merged 8 commits into from
Dec 4, 2022

Conversation

nilsleh
Copy link
Collaborator

@nilsleh nilsleh commented Nov 27, 2022

Change the argument name for the classifcation task according to comment in #854.

I put this in a separate PR because with classification there are quiet a lot of file changes.

@github-actions github-actions bot added scripts Training and evaluation scripts testing Continuous integration testing trainers PyTorch Lightning trainers labels Nov 27, 2022
evaluate.py Show resolved Hide resolved
evaluate.py Outdated Show resolved Hide resolved
torchgeo/trainers/classification.py Show resolved Hide resolved
torchgeo/trainers/utils.py Outdated Show resolved Hide resolved
@adamjstewart adamjstewart added the backwards-incompatible Changes that are not backwards compatible label Nov 27, 2022
@adamjstewart adamjstewart added this to the 0.4.0 milestone Nov 27, 2022
adamjstewart
adamjstewart previously approved these changes Nov 27, 2022
@adamjstewart adamjstewart merged commit b290f7b into microsoft:main Dec 4, 2022
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
* name change

* fix failing test

* expose all available timm models

* chmod

* imagenet pretrained flag

* remove extra

* docstring list_models
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-incompatible Changes that are not backwards compatible scripts Training and evaluation scripts testing Continuous integration testing trainers PyTorch Lightning trainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants