Skip to content
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

dicom structured report prefix #4806

Conversation

thewtex
Copy link
Member

@thewtex thewtex commented Aug 16, 2024

The dsr2html tool uses a hardcoded literal string as a hyperlink prefix for refrenced composite objects (images, waveforms, etc.) in rendering the Key Object Selection documents.

Replace this literal with a string argument that will be passed by a calling application / function.

Pushed upstream: DCMTK/dcmtk#108

The dsr2html tool uses a hardcoded literal string as a hyperlink prefix for refrenced composite objects (images, waveforms, etc.) in rendering the Key Object Selection documents.

Replace this literal with a string argument that will be passed by a calling application / function.

Pushed upstream: DCMTK/dcmtk#108
@github-actions github-actions bot added the area:ThirdParty Issues affecting the ThirdParty module label Aug 16, 2024
@thewtex thewtex requested a review from jadh4v August 16, 2024 17:14
For DICOM SEG, parametric maps.
@github-actions github-actions bot added the type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots label Aug 16, 2024
Copy link
Member

@jadh4v jadh4v left a comment

Choose a reason for hiding this comment

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

LGTM

@thewtex thewtex merged commit aabba2f into InsightSoftwareConsortium:master Aug 23, 2024
12 checks passed
@thewtex thewtex deleted the dicom-structured-report-prefix branch August 23, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ThirdParty Issues affecting the ThirdParty module type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants