-
Notifications
You must be signed in to change notification settings - Fork 67
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
Update @itk-wasm/dicom for new SEG pipeline #653
Conversation
✅ Deploy Preview for volview-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
34acfe9
to
650b9d2
Compare
a752baf
to
e555bee
Compare
Just to make sure there is no misunderstanding here.
@jadh4v - one of the above would be a good sample to include in testing in InsightSoftwareConsortium/ITK-Wasm#1208. As always, to download the images in the above example, you can use
Note that to see overlapping segments in the above I used OHIF v2. The current OHIF v3 cannot display overlapping segments as of yet, see OHIF/Viewers#3496. |
I did misunderstood. Thanks for the clarification and files. Changed to do the wrong thing better. That is: just using the first output file and popping a warning. |
mergeSegments
option to true, pull first output file/component for use as segment group, and pop a warning.closes #639
Suggest we just put these in an issue and not do in this PR: