enh: add notebook demonstrating access to data from Imaging Data Commons#2063
enh: add notebook demonstrating access to data from Imaging Data Commons#2063ericspod merged 10 commits intoProject-MONAI:mainfrom
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (1)
⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
You can disable this status message by setting the Use the checkbox below for a quick retry:
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)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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. 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
Co-authored-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com> Signed-off-by: Andrey Fedorov <fedorov@bwh.harvard.edu>
c6c45de to
543253c
Compare
Signed-off-by: Andrey Fedorov <andrey.fedorov@gmail.com>
|
@ericspod thank you for the review. I addressed your suggestions, thank you. I do not see any way how I can merge this, can you please do it? I can only close or comment. |
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