-
Notifications
You must be signed in to change notification settings - Fork 56
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
suggestion to avoid issue when multiple types of files are in the same folder #51
Comments
This is a really good idea! Perhaps instead of hardcoding it to "CT", there could be an option which defaults to CT, but is changable to other modalities. It could even be set with SOPClassUID to avoid overlapping uses of Modality? |
Thanks for reaching out @EmmaSkarsoe, and good call @mathiser. I'm not sure when I can work on that but that seems like a good solution. |
Dear @asim-shrestha Are you open to pull requests? |
Yes of course! Please feel free to submit any of your desired changes :) |
Hi rt_Utils authors,
Thanks for a great tool !
I've experienced some problems, when I am running the add_roi function.
I got the following error-message: Mask must have the same number of layers (In the 3rd dimension) as input series. Expected 130, got 129.
The problem arises, because my folder contains both RT structs, CT scans, RT dose and RT plan. The RT dose file do also have a 'pixel_array' attribute.
I changed a line in the function: load_dcm_images_from_path
from
if hasattr(ds, "pixel_array"):
series_data.append(ds)
to
if ds.Modality == 'CT':
series_data.append(ds)
It solved my problem. :)
The text was updated successfully, but these errors were encountered: