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

dcmtk expects monotonically increasing SegmentNumber #50

Open
razorx89 opened this issue May 15, 2023 · 0 comments · May be fixed by #52
Open

dcmtk expects monotonically increasing SegmentNumber #50

razorx89 opened this issue May 15, 2023 · 0 comments · May be fixed by #52
Labels
bug Something isn't working

Comments

@razorx89
Copy link
Owner

Currently, only segments with a positive mask get copied to the SegmentSequence:

writer_utils.copy_segmentation_template(
target=result,
template=self._template,
segments=labels_to_process,
skip_missing_segment=self._skip_missing_segment,
)

However, the DICOM standard says "Segment Number (0062,0004) shall be unique within each Instance, start at a value of 1, and increase monotonically by 1.". Dcmtk correctly enforces this and throws an error.

@razorx89 razorx89 added the bug Something isn't working label May 15, 2023
@razorx89 razorx89 linked a pull request May 16, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant