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

Load checkpoints with eval_setup from different folder from training #3485

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kerrj
Copy link
Collaborator

@kerrj kerrj commented Oct 16, 2024

Previously, if one ran ns-train from a directory, then any call to eval_setup would have to be called from a directory where the relative layout of the outputs folder had to match training. I changed this to take into account the full path to the config.yml file in eval_setup so that it loads checkpoints folders not equivalent to the training folder.

For example, running this will fail:

ns-train ...
cd ..
ns-viewer --load-config nerfstudio/path/to/config.yml

There still needs to be some work to make data path agnostic to the running dir, but this so far at least makes it agnostic for the checkpoint folder.

One option is making data paths absolute, though this would break if you try to send checkpoints+data to other people, or move the original data dir around.

@kerrj kerrj closed this Oct 16, 2024
@kerrj kerrj reopened this Oct 16, 2024
@kerrj kerrj marked this pull request as draft October 16, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant