Skip to content

Update Colab demo notebook#128

Merged
C-Achard merged 11 commits into
mainfrom
cy/colab-update
May 22, 2026
Merged

Update Colab demo notebook#128
C-Achard merged 11 commits into
mainfrom
cy/colab-update

Conversation

@C-Achard
Copy link
Copy Markdown
Collaborator

@C-Achard C-Achard commented May 21, 2026

Content

  • Fixes mismatched URL for "Open in Colab" button
  • Switch install to uv
  • Tested successfully on latest Colab

@C-Achard C-Achard self-assigned this May 21, 2026
@C-Achard C-Achard added enhancement New feature or request fix Something needs to be or has been fixed colab labels May 21, 2026
@C-Achard C-Achard requested a review from Copilot May 21, 2026 11:25
@C-Achard C-Achard linked an issue May 21, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Colab inference demo notebook to modernize installation steps and correct the “Open in Colab” entry point.

Changes:

  • Updates the “Open in Colab” badge to point to the AdaptiveMotorControlLab GitHub org.
  • Switches installation from pip to uv (including installing uv in the notebook).
  • Keeps cloning the repo in-notebook for demo assets.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread notebooks/Colab_inference_demo.ipynb Outdated
Comment thread notebooks/Colab_inference_demo.ipynb Outdated
Comment thread notebooks/Colab_inference_demo.ipynb Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread notebooks/Colab_inference_demo.ipynb Outdated
Update Colab_inference_demo.ipynb to ensure demo data and improve setup flow. Add REPO_DIR/DEMO_FILE/REPO_URL constants, import time, and a run() print fix. Introduce ensure_demo_data() to clone the CellSeg3D repo and verify the expected example file, removing previous ad-hoc cloning logic. Track whether the package was installed with installed_package, improve user messages about runtime restart, add a short sleep before killing the process, and print a concise completion message when setup is already satisfied.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread notebooks/Colab_inference_demo.ipynb Outdated
Comment thread notebooks/Colab_inference_demo.ipynb
@C-Achard C-Achard marked this pull request as ready for review May 21, 2026 13:20
@C-Achard C-Achard requested a review from MMathisLab May 21, 2026 13:21
@C-Achard
Copy link
Copy Markdown
Collaborator Author

@C-Achard C-Achard merged commit 406c8b1 into main May 22, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

colab enhancement New feature or request fix Something needs to be or has been fixed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Colab inference demo incompatible with current MONAI and NumPy

2 participants