Skip to content

ENH: Add data download instructions to readme quickstart - #112

Merged
aylward merged 1 commit into
Project-MONAI:mainfrom
aylward:README-downloaddata
Aug 6, 2026
Merged

ENH: Add data download instructions to readme quickstart#112
aylward merged 1 commit into
Project-MONAI:mainfrom
aylward:README-downloaddata

Conversation

@aylward

@aylward aylward commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 6, 2026 00:33
@aylward
aylward merged commit 07a8873 into Project-MONAI:main Aug 6, 2026
6 of 7 checks passed
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@aylward, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 52 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6100d4f5-48a7-4fa2-b1e1-0fc3d3efadff

📥 Commits

Reviewing files that changed from the base of the PR and between de73477 and 977a873.

📒 Files selected for processing (2)
  • README.md
  • docs/quickstart.rst

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@aylward
aylward deleted the README-downloaddata branch August 6, 2026 00:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds explicit dataset download commands to the user-facing quickstart guidance so new users can run the heart/lung tutorials with the expected sample data in the correct repo-relative locations.

Changes:

  • Update README “Quick Start” tutorial-clone snippet to include downloading Slicer-Heart-CT.
  • Expand docs/quickstart.rst dataset download section with per-tutorial dataset calls and clearer wording for Tutorial 1.

Reviewed changes

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

File Description
README.md Adds a concrete physiotwin4d-download-data command before running Tutorial 1 and explains repo-root download location.
docs/quickstart.rst Adds labeled download commands for multiple tutorial datasets and clarifies Tutorial 1 prerequisites.

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

Comment thread README.md
Comment on lines +101 to +103
Run the download from the top level of the clone: the tutorials resolve their
inputs against the repository root, so downloading elsewhere puts the data
where they will not find it.
Comment thread docs/quickstart.rst
Comment on lines +61 to 62
# Tutorials 1, 3 and 4 (heart)
physiotwin4d-download-data Slicer-Heart-CT --directory data/Slicer-Heart-CT
@codecov

codecov Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.60%. Comparing base (de73477) to head (977a873).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   36.60%   36.60%           
=======================================
  Files          72       72           
  Lines        8510     8510           
=======================================
  Hits         3115     3115           
  Misses       5395     5395           
Flag Coverage Δ
integration-tests 36.40% <ø> (?)
unittests 36.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants