Skip to content

Conversation

@carluri
Copy link
Collaborator

@carluri carluri commented Jan 24, 2026

Extracted DICOM file organization logic from DICOMReceiver into a standalone dicom_organizer module, achieving clean separation of concerns and eliminating code duplication.

Copy link

Copilot AI left a 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 DICOMReceiver to 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.

carluri and others added 6 commits January 24, 2026 16:41
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>
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