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

Updating the script for downloading data #968

Merged
merged 7 commits into from
Nov 26, 2022
Merged

Conversation

jake-austin
Copy link
Contributor

No description provided.

Copy link
Contributor

@tancik tancik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes will break everywhere the download data script is currently being used. These also need to be updated. From memory this include documentation and the colab.

Copy link
Collaborator

@ethanweber ethanweber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking better, but there are a lot of docs that need to updated still. For example, commands like ns-download-data --dataset=nerfstudio --capture=poster. Also looks like you renamed --capture to --capture-name, which will have to be updated in the docs as well. I'm okay with the new name since it's more verbose.

Copy link
Collaborator

@ethanweber ethanweber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but that'd be great to update the PR description first.

@jake-austin jake-austin merged commit 1eae083 into main Nov 26, 2022
@jake-austin jake-austin deleted the download-script-updates branch November 26, 2022 05:40
tancik pushed a commit to dozeri83/nerfstudio that referenced this pull request Jan 20, 2023
* Updating the script for downloading data

* Making update to config structure

* Updating documentation and colab

* Adding back the save directory

* Pushing an all flag for nerfstudio

* forgot to save some previous edits
chris838 pushed a commit to chris838/nerfstudio that referenced this pull request Apr 22, 2023
* Updating the script for downloading data

* Making update to config structure

* Updating documentation and colab

* Adding back the save directory

* Pushing an all flag for nerfstudio

* forgot to save some previous edits
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.

4 participants