Skip to content

Conversation

@edyoshikun
Copy link
Member

This PR adds a quickstart demo for the VCP

# %%
# Validate installation
# !viscy --help

Copy link
Collaborator

@KRCora KRCora Nov 26, 2025

Choose a reason for hiding this comment

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

Suggested change
### Download example data and model checkpoint
Estimated download time: XX

Copy link
Collaborator

Choose a reason for hiding this comment

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

@edyoshikun FYI this step is taking a long time... I wasn't able to run it on Colab's free tier (ran out of disk space). Trying on Colab Enterprise

Copy link
Member Author

Choose a reason for hiding this comment

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

Addressed in 3a71560

Comment on lines 83 to 96
"\"\"\"\n",
"Setup\n",
"\n",
"The commands below will install the required packages and download the example dataset and model checkpoint.\n",
"\n",
"Setup notes:\n",
"\n",
"- **Setting up Google Colab**: To run this quickstart guide using Google Colab, choose the 'T4' GPU runtime from the \"Connect\" dropdown menu in the upper-right corner of this notebook for faster execution.\n",
"Using a GPU significantly speeds up running model inference, but CPU compute can also be used.\n",
"\n",
"- **Setting up local environment**: The commands below assume a Unix-like shell with `wget` installed. On Windows, the files can be downloaded manually from the URLs.\n",
"On Windows, the files can be downloaded manually from the URLs.\n",
"\n",
"\"\"\"\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think something's wrong with the formatting here (looking at the rendered notebook shows extra quotes):

Suggested change
"\"\"\"\n",
"Setup\n",
"\n",
"The commands below will install the required packages and download the example dataset and model checkpoint.\n",
"\n",
"Setup notes:\n",
"\n",
"- **Setting up Google Colab**: To run this quickstart guide using Google Colab, choose the 'T4' GPU runtime from the \"Connect\" dropdown menu in the upper-right corner of this notebook for faster execution.\n",
"Using a GPU significantly speeds up running model inference, but CPU compute can also be used.\n",
"\n",
"- **Setting up local environment**: The commands below assume a Unix-like shell with `wget` installed. On Windows, the files can be downloaded manually from the URLs.\n",
"On Windows, the files can be downloaded manually from the URLs.\n",
"\n",
"\"\"\"\n",
"### Setup\n",
"\n",
"The commands below will install the required packages and download the example dataset and model checkpoint.\n",
"\n",
"Setup notes:\n",
"\n",
"- **Setting up Google Colab**: To run this quickstart guide using Google Colab, choose the 'T4' GPU runtime from the \"Connect\" dropdown menu in the upper-right corner of this notebook for faster execution.\n",
"Using a GPU significantly speeds up running model inference, but CPU compute can also be used.\n",
"\n",
"- **Setting up local environment**: The commands below assume a Unix-like shell with `wget` installed. On Windows, the files can be downloaded manually from the URLs.\n",
"On Windows, the files can be downloaded manually from the URLs.\n",

Copy link
Member Author

Choose a reason for hiding this comment

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

addressed this in ab92b74

"cell_marker": "\"\"\""
},
"source": [
"# Prerequisites\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
"# Prerequisites\n",
"## Prerequisites\n",

Copy link
Member Author

Choose a reason for hiding this comment

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

addressed 03d71f0

"axes[1].set_ylabel(\"PHATE 2\")\n",
"\n",
"plt.tight_layout()\n",
"plt.show()"
Copy link
Contributor

Choose a reason for hiding this comment

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

This cell does not work for me, due to an issue with not finding the extracted_inf_state.csv file.

FileNotFoundError: [Errno 2] No such file or directory: 'extracted_inf_state.csv'

Did we fail to download this file in an earlier step?

Copy link
Member Author

Choose a reason for hiding this comment

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

I figured out the bug. 1) I forgot to write the wget for downloading 2) the permission for the file were set to 750. Thanks!

Copy link
Member Author

Choose a reason for hiding this comment

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

ab92b74 the notebook now reflects the download of this data and the csv file needed to run the rest of the exercise

@edyoshikun edyoshikun requested a review from melissawm November 28, 2025 14:18
@melissawm
Copy link
Contributor

melissawm commented Nov 28, 2025

Hi @edyoshikun - thanks for this!

Last comments:

  • When running datamodule.setup("predict"), the notebook asks for access to Kaggle credentials. This should probably be noted as a prerequisite to the quickstart.
  • When running the VisCyTrainer step, I get an error:
WARNING: PHATE computation failed: PHATE is not available. Install with: pip install viscy[phate]
WARNING:lightning.pytorch:PHATE computation failed: PHATE is not available. Install with: pip install viscy[phate]

After installing phate, the notebook runs great 🎉

Copy link
Contributor

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Looks great! When you merge this, we can use add the notebook URL to the VCP so it's linked from there. Thank you!

@edyoshikun edyoshikun merged commit 6bc7771 into main Dec 1, 2025
4 checks passed
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.

5 participants