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.

Sign up for GitHub

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

Jump to bottom

Tutorials: trainers require optional datasets #759

Merged
merged 1 commit into from
Sep 5, 2022
Merged

Tutorials: trainers require optional datasets #759

merged 1 commit into from
Sep 5, 2022

Conversation

adamjstewart
Copy link
Collaborator

Separated this into a separate PR.

@adamjstewart adamjstewart added this to the 0.3.1 milestone Sep 5, 2022
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 5, 2022
@adamjstewart
Copy link
Collaborator Author

Tested this tutorial locally. It still fails, but the reason it fails is because we currently run pip install torchgeo, which installs 0.3.0, which installs radiant-mlhub 0.5.2, which doesn't work. Once we release, pip install torchgeo will install 0.3.1, which will install radiant-mlhub 0.4.1, which will work. So this mystery has been solved.

@adamjstewart adamjstewart merged commit 0681887 into main Sep 5, 2022
@adamjstewart adamjstewart deleted the docs/trainers branch September 5, 2022 19:18
adamjstewart added a commit that referenced this pull request Sep 5, 2022
@adamjstewart
Tutorials: trainers require optional datasets (#759)
7bbd68c
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
@adamjstewart
Tutorials: trainers require optional datasets (microsoft#759)
a0218f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calebrob6 calebrob6 calebrob6 approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
0.3.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@adamjstewart @calebrob6