-
Notifications
You must be signed in to change notification settings - Fork 3
Improve functionality to organize long lists of DICOM data into patient, cycle, timepoint #102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
…w module for better post processing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Refactors DICOM post-processing so DICOM file organization (patient → cycle → timepoint) is implemented in a standalone dicom_organizer module and reused by DICOMReceiver.
Changes:
- Extracted and generalized cycle/timepoint organization into
pytheranostics/dicomtools/dicom_organizer.py. - Updated
DICOMReceiverto call the shared organizer for auto-organize behavior. - Added Sphinx documentation/tutorial page for the new organizer utilities and exposed them at the package level.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
pytheranostics/dicomtools/dicom_receiver.py |
Switches auto-organize logic to call the new shared organizer API. |
pytheranostics/dicomtools/dicom_organizer.py |
New module implementing recursive DICOM organization and summarization utilities. |
pytheranostics/dicomtools/__init__.py |
Exposes organizer utilities as package-level imports. |
docs/source/tutorials/index.rst |
Adds the new DICOM organization tutorial to the docs TOC. |
docs/source/tutorials/dicom_organization/dicom_organization.rst |
New tutorial documenting usage, parameters, and examples. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
docs/source/tutorials/dicom_organization/dicom_organization.rst
Outdated
Show resolved
Hide resolved
docs/source/tutorials/dicom_organization/dicom_organization.rst
Outdated
Show resolved
Hide resolved
docs/source/tutorials/dicom_organization/dicom_organization.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Extracted DICOM file organization logic from
DICOMReceiverinto a standalonedicom_organizermodule, achieving clean separation of concerns and eliminating code duplication.