haphantran
commented
3 months ago LGTM, Thanks! Just one thing, would it be useful to add a few comments for the error messages so it is clear what they are for?
Good ideas. It might be too complicated to catch all kind of errors, but I will try to see if we can be more specific in the next PR.
This change is to support Dicome Files by allow user to select the mask and extract the mask from the dicom files automatically. It will only extract the selected mask to save on process time and storage. Trello https://trello.com/c/aV2bniwQ/8-high-priority-support-loading-of-dicom-structures-loading-dcm-structure-found-with-folder-dcmstructlung4d Here is the demo (this is unlisted so only people with the link can view) https://youtu.be/s9YfhXYzAOs