rt-utils icon indicating copy to clipboard operation
rt-utils copied to clipboard

suggestion to avoid issue when multiple types of files are in the same folder

Open EmmaSkarsoe opened this issue 3 years ago • 4 comments

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. :)

EmmaSkarsoe avatar May 12 '22 08:05 EmmaSkarsoe

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?

mathiser avatar Aug 31 '22 06:08 mathiser

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.

asim-shrestha avatar Sep 27 '22 04:09 asim-shrestha

Dear @asim-shrestha It's so great there is some activity on the repo again!

Are you open to pull requests?

mathiser avatar Sep 27 '22 05:09 mathiser

Yes of course! Please feel free to submit any of your desired changes :)

asim-shrestha avatar Sep 27 '22 23:09 asim-shrestha