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

CLI script for test #2082

Closed
yichiac opened this issue May 24, 2024 · 2 comments · Fixed by #2091
Closed

CLI script for test #2082

yichiac opened this issue May 24, 2024 · 2 comments · Fixed by #2091
Labels
documentation Improvements or additions to documentation good first issue A good issue for a new contributor to work on
Milestone
0.5.3

Comments

@yichiac
Copy link
Contributor

yichiac commented May 24, 2024

Issue

The current documentation for testing is:
torchgeo test --config config.yaml --trainer.ckpt_path=...

It would return:
error: Unrecognized arguments: --trainer.ckpt_path=...

Fix

It shuold be
torchgeo test --config config.yaml ckpt_path=...
@yichiac yichiac added the documentation Improvements or additions to documentation label May 24, 2024
@adamjstewart
Copy link
Collaborator

You are correct. Want to submit a PR?

@adamjstewart adamjstewart added this to the 0.5.3 milestone May 25, 2024
@adamjstewart adamjstewart added the good first issue A good issue for a new contributor to work on label May 25, 2024
@TropicolX TropicolX mentioned this issue May 27, 2024
Merged
@TropicolX
Copy link
Contributor

Hi @adamjstewart, I submitted a PR for this here: #2091

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation good first issue A good issue for a new contributor to work on
Projects
None yet
Milestone
0.5.3
Development

Successfully merging a pull request may close this issue.

Update CLI script for test TropicolX/torchgeo
3 participants
@adamjstewart @yichiac @TropicolX