Skip to content

Update instructions to run Jupyter notebooks#71

Merged
smoreinis merged 2 commits into
mainfrom
stas/notebook-docs
Nov 10, 2025
Merged

Update instructions to run Jupyter notebooks#71
smoreinis merged 2 commits into
mainfrom
stas/notebook-docs

Conversation

@smoreinis

Copy link
Copy Markdown
Contributor

Adding explicit uv command to run these once the notebook tool has been installed.

@smoreinis
smoreinis requested a review from a team as a code owner November 10, 2025 20:00
Copilot AI review requested due to automatic review settings November 10, 2025 20:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Jupyter notebook documentation to include explicit installation and launch instructions. The changes clarify the setup process by adding the notebook installation step and providing the command to run notebooks.

Key Changes:

  • Added Jupyter Notebook installation instruction to prerequisites
  • Added explicit command to launch Jupyter notebooks using uv

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

- **Code snippets** demonstrating both streaming and non-streaming patterns

Simply open the notebook and run the cells to start testing your agent immediately. The examples below explain what's in the notebook and how to customize it for your needs.
Simply open the notebook and run the cells to start testing your agent immediately - you can launch it with `uv run jupyter notebook dev.ipynb`.

Copilot AI Nov 10, 2025

Copy link

Choose a reason for hiding this comment

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

The command uv run jupyter notebook dev.ipynb is inconsistent with the installation method shown in line 16. Since Jupyter was installed using uv tool install notebook, it should be launched with uv tool run notebook dev.ipynb or just notebook dev.ipynb (if the tool's bin directory is in PATH), not uv run.

Suggested change
Simply open the notebook and run the cells to start testing your agent immediately - you can launch it with `uv run jupyter notebook dev.ipynb`.
Simply open the notebook and run the cells to start testing your agent immediately - you can launch it with `uv tool run notebook dev.ipynb`.

Copilot uses AI. Check for mistakes.
@smoreinis
smoreinis merged commit 2841a08 into main Nov 10, 2025
6 checks passed
@smoreinis
smoreinis deleted the stas/notebook-docs branch November 10, 2025 20:03
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.

3 participants