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

Update colab for new viewer #2787

Closed
wants to merge 6 commits into from
Closed

Update colab for new viewer #2787

wants to merge 6 commits into from

Conversation

ginazhouhuiwu
Copy link
Contributor

@ginazhouhuiwu ginazhouhuiwu commented Jan 18, 2024

Changes:

  • deprecate old viewer elements in notebook
  • use conda for colmap installation

TODO: Viser share link doesn't seem to be printing properly. ns-train cell sometimes stops executing after saving the configs.

},
"outputs": [],
"source": [
"#@markdown <h1>Install Nerfstudio and Dependencies (~8 min)</h1>\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this still take around 8min?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will check, it took kind of long for me.

@yimingc
Copy link

yimingc commented Jan 21, 2024

Thanks for trying fix this.

I got error when I ran in colab on your branch tip
image

@ginazhouhuiwu
Copy link
Contributor Author

Hey! Thanks for letting me know the errors! Unfortunately I'm still working on debugging the training portion in this pr draft... But your error seems to point to a tinycudann installation error, which I can't recreate from my end. Can you let me know if that was properly installed from running the first cell in the notebook?

@AntonioMacaronio
Copy link
Contributor

AntonioMacaronio commented Jan 25, 2024

This is still an error - on line 14 inside the cell "Install Nerfstudio and Dependencies (~8 min)":
!gdown "https://drive.google.com/u/0/uc?id=15WngFRNar_b8CaPR5R-hvQ3eAnlyk_SL&confirm=t"

tinycudann is able to be installed perfectly, but the drive link for the colmap package is not sharable (even with the confirm=t tag on the url for the large gdrive files). Perhaps that is why it is not reproducible?

I see in this PR there are new fixes with pip install condalab - i added this to mine but the training cell will still say "Col

I also just tried the fix noted here: #2406 (comment) but I am still running into various versioning errors when running the following "Downloading and Processing Data" cell

},
"outputs": [],
"source": [
"#@markdown <h1>Install Nerfstudio and Dependencies (~8 min)</h1>\n",
"#@markdown <h3>Install Nerfstudio and Dependencies (~8 min)</h3>\n",
"\n",
"%cd /content/\n",
"!pip install --upgrade pip\n",
"!pip install torch==2.0.1+cu118 torchvision==0.15.2+cu118 --extra-index-url https://download.pytorch.org/whl/cu118\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"!pip install torch==2.0.1+cu118 torchvision==0.15.2+cu118 --extra-index-url https://download.pytorch.org/whl/cu118\n",
"!pip install torch==2.0.1+cu118 torchvision==0.15.2+cu118 torchaudio==2.0.2+cu118 --extra-index-url https://download.pytorch.org/whl/cu118\n",

I also needed to pin the version of torchaudio to get training to start on Google Colab. I only tested it on Google Colab and as of today.

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.

None yet

5 participants