enh: add notebook demonstrating access to data from Imaging Data Commons#2063
enh: add notebook demonstrating access to data from Imaging Data Commons#2063fedorov wants to merge 8 commits intoProject-MONAI:mainfrom
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughREADME updated to add an ChangesSingle cohesive update (docs + runner plumbing)
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
for more information, see https://pre-commit.ci
Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
…delines - Fix license header: use MONAI Consortium copyright, correct format with trailing double spaces and indentation, moved to top of first cell - Move all imports (os, sys, itkwasm_dicom) into Setup imports cell; simplify Setup environment cell to pip install only - Add README.md entry for idc_dataset under Modules section - Add idc_dataset to doesnt_contain_max_epochs and skip_run_papermill in runner.sh Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 391: Remove the garbled artifact line containing "ECHO�� �����Ǿ� �ֽ��ϴ�."
from the README; simply delete that exact text (the stray ECHO plus non-ASCII
characters) so the file no longer contains the corrupted Windows echo artifact,
then verify the README renders correctly (preview or lint) to ensure no other
non-ASCII artifacts remain.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: d9a7dea9-c32f-4a1f-9fd0-f3000898b7f8
📒 Files selected for processing (3)
README.mdmodules/idc_dataset.ipynbrunner.sh
Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
for more information, see https://pre-commit.ci
Description
This notebook demonstrates how to load images and segmentations from Imaging Data Commons.
Checks
./figurefolder./runner.sh -t <path to .ipynb file>Summary by CodeRabbit