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 sort rebase #1197

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

thewtex
Copy link
Member

@thewtex thewtex commented Aug 13, 2024

  • feat(image-sets-normalization): add new pipeline
  • fix(image-sets-normalization): update codegened files for new pipeline
  • feat(image-sets-normalization): Sort DICOMs
  • feat(image-sets-normalization): read DICOM tags
  • fix(gdcmDiscriminateVolume): Remove prints
  • chore(test_image_sets_normalization): fix one series test
  • refactor(image-sets-normalization): rename output to image-sets
  • refactor(image-sets-normalization): revamp data structures
  • fix(image-sets-normalization): fix sorting of files in series
  • feat(image-sets-normalization): make tag keys english
  • fix(image-sets-normalization): check more tags when grouping volumes
  • feat(image-sets-normalization): array and number types for tag values
  • chore(dicom): add mr test for image sets normalization
  • WIP feat(image-sets-normalization): convert to UTF8 from character set
  • WIP fix(image-sets-normalization): remove trailing space on tag values
  • WIP fix(image-sets-normalization): fallback to tag hex key if private or empty
  • WIP feat(image-sets-normalization): add group by options

thewtex and others added 18 commits August 27, 2024 09:24
Change CmakeLists to build image-sets-normalization for WASI.

Fix wasi/__init__ to expose image_sets_normalization.

Fix generated file import of image in test
Use GDCM discriminate volume example class.
 Use DICOMTagReader to get all tags in files
Also, remove import of "itkGDCMImageIO.h" by extracting CharStringToUTF8Converter
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