Skip to content

Make the output DICOM SR instance part of the original study #394

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

Merged
merged 2 commits into from
Nov 22, 2022

Conversation

MMelQin
Copy link
Collaborator

@MMelQin MMelQin commented Nov 12, 2022

The Breast Density Classification example app generates DICOM SR instance as if in a new study. This by itself is supported if the original input DICOM is not available, e.g. in cases where input images are non-DICOM files.

The example as created expects to load and parse input DICOM files, so it's expected that the generated DICOM SR instance will copy the study level tags of the input, in order to be of the same study.

Added also is the sample series selection rule, which will only select MG series and "Primary" Image Type (note that the sample inputs have "Derived/Primary" Image Type due to the 2D instances are projection images from the 3D Tomo Mammo acquisition)

Signed-off-by: M Q [email protected]

@MMelQin MMelQin self-assigned this Nov 12, 2022
@MMelQin MMelQin added the enhancement New feature or request label Nov 12, 2022
@MMelQin MMelQin requested a review from vikashg November 12, 2022 03:01
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Collaborator

@vikashg vikashg left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@vikashg vikashg merged commit 3298b7d into main Nov 22, 2022
@MMelQin MMelQin deleted the mqin/breast_density branch March 17, 2023 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[IMP] Breast Density Classification app to make output DICOM SR instance part of the original study
2 participants