-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[WIP] 141 add dicom loader #200
Closed
mhubii
wants to merge
45
commits into
Project-MONAI:135-add-DICOM-loader
from
mhubii:135-add-DICOM-loader
Closed
Changes from 1 commit
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
f5ccf2a
Adding rotation transform. (#126)
madil90 f39e8c1
108-resize (#125)
wyli 28ad305
113-gaussian-noise (#139)
wyli 7f29477
133 dataset and example with dict transforms (#134)
wyli 445a568
107-zoom (#138)
madil90 db05942
Fix gpu zoom errors. (#149)
madil90 2bda586
58-implement-transform-adaptor-2: Re-adding work via new branch as th…
atbenmurray b850167
update doc api deps (#158)
wyli 03bebf5
152-random-zoom (#153)
madil90 1e230ed
Adding RandomFlip. (#154)
madil90 7304182
112 combining spatial transforms (#131)
wyli 39b2cb2
144 loadnifti transform common dataset (#145)
Nic-Ma 35da65c
spacing and orientation; revise transforms cropping and zooming (#162)
wyli 5c49f8f
156 event handlers support arbitrary format (#159)
Nic-Ma d166f6a
152-random-rotate (#155)
madil90 5b1f258
add dictionary-based wrapper random spatial transforms (#166)
wyli 3a8f99c
160 develop TensorBoard event handler (#161)
Nic-Ma e40917f
Adding dict-based and random spatial transforms. (#163)
madil90 204e42d
two stages ci (#172)
wyli 895e0f2
165 update check all examples (#171)
Nic-Ma c40f739
179 add DeleteKeys transform (#180)
Nic-Ma 216298c
176 revise docs (#177)
wyli c2b12d3
[DLMED] fix typos in documentations (#185)
Nic-Ma 3568c2b
181 use MONAI in pytorch medical segmentation program (#182)
Nic-Ma b86b63a
added setuptools support
mhubii 3781679
updated README
mhubii 2b9ab58
removed relative paths
mhubii 4762c6f
changed form
mhubii 40a06b0
fixes pull request config (#194)
wyli e7bb1c1
Merge branch 'master' into master
wyli 1e55e23
exchanged raw install by pip install with folder flag
mhubii e4b3c36
Merge branch 'master' of https://github.com/mhubii/MONAI
mhubii 9f10654
188 setuptools (#193)
mhubii e7b6564
188 setup docker image (#192)
wyli d191b09
191 develop determinism integration test (#195)
Nic-Ma 85efd81
Merge branch 'master' into 135-add-DICOM-loader
mhubii 8de36ae
Merge remote-tracking branch 'upstream/master'
mhubii a9521bf
solved merge conflicts
mhubii 732befd
Merge branch 'master' into 135-add-DICOM-loader
mhubii 469a136
added SimpleITK and pydicom to dependencies
mhubii f7c0205
fixed typo
mhubii d1a21d6
added LoadDICOM and dictionary loader
mhubii 21ae759
added utility functions for dicom handling
mhubii 5c0a207
added tests on pydicom's sample data
mhubii 805c214
added spaces to inline comment
mhubii File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your update here.
Actually, I think you can treat this converter tool as a temporary reference code and move all the data converter logic into other transforms(or new transforms) after "LoadDICOM".
When all logic is ready in transforms, we can delete this tool and users can use transforms to load DICOM directly.
Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @mhubii ,
Do you want to merge this PR first to my previous DICOM PR first and work on that PR directly?
Thanks.