Skip to content

docs: quiet tutorial dependency installation - #726

Open
zywind wants to merge 1 commit into
yunfeng/healthcare-readmissions-tutorialfrom
yunfeng/quiet-tutorial-installs
Open

docs: quiet tutorial dependency installation#726
zywind wants to merge 1 commit into
yunfeng/healthcare-readmissions-tutorialfrom
yunfeng/quiet-tutorial-installs

Conversation

@zywind

@zywind zywind commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Validate notebook JSON and install-cell Bash syntax.
  • Run the uv install commands for the core package, datasets, kagglehub, and xgboost.
  • Verify the child diff contains only --quiet changes.

Signed-off-by: Yunfeng Zhang <yunzhang@nvidia.com>
@zywind
zywind requested a review from a team as a code owner August 20, 2026 15:06
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 2251539e-418f-462d-b408-c3a6e889bff5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds --quiet to the uv pip install and pip install commands in four tutorial notebooks so routine dependency-installation output is suppressed while errors remain visible.

  • Quiets Safe Synthesizer installation in the differential-privacy, healthcare-readmissions, introductory, and financial time-series tutorials.
  • Also quiets installation of tutorial-specific dependencies such as datasets and kagglehub.

Confidence Score: 4/5

The PR appears safe to merge functionally, but its modified notebook cells should use the repository-required uv-only installation workflow.

The quiet options are valid in the changed command positions and preserve installation failures, leaving only the non-blocking repository-guidance violation in the raw-pip fallback branches.

Files Needing Attention: docs/tutorials/differential-privacy.ipynb, docs/tutorials/healthcare-hospital-readmissions.ipynb, docs/tutorials/safe-synthesizer-101.ipynb, docs/tutorials/time-series-financial-transactions.ipynb

Important Files Changed

Filename Overview
docs/tutorials/differential-privacy.ipynb Adds quiet flags to Safe Synthesizer and KaggleHub installation, while retaining a raw-pip fallback contrary to repository guidance.
docs/tutorials/healthcare-hospital-readmissions.ipynb Quiets Safe Synthesizer and XGBoost installation, with the same raw-pip fallback pattern.
docs/tutorials/safe-synthesizer-101.ipynb Adds quiet flags to core and datasets installation commands, with the same raw-pip fallback pattern.
docs/tutorials/time-series-financial-transactions.ipynb Quiets the Safe Synthesizer installation command, with the same raw-pip fallback pattern.

Reviews (1): Last reviewed commit: "docs: quiet tutorial dependency installa..." | Re-trigger Greptile

Comment on lines +54 to +55
" --extra-index-url https://wheels.vllm.ai/0.26.0/cu129 --quiet\n",
" pip install kagglehub --quiet\n",

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.

P2 Raw pip fallback remains

When uv is unavailable, this modified install cell invokes raw pip install, contrary to the repository's required uv-only dependency workflow; the same pattern remains in the other three modified notebooks.

Context Used: AGENTS.md (source)

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant