[BUG] DICOM import does not import RTdose - Githubissues

e0404 / matRad

An open source multi-modality radiation treatment planning sytem developed by e0404 @ DKFZ

http://www.matRad.org

Other

223 stars 176 forks source link

[BUG] DICOM import does not import RTdose #558

Open pietemma opened 2 years ago

pietemma commented 2 years ago

Hello am using the matRad standalone, when i import DICOM data it only imports Rt structure and CT but doesn't import the plan and the doses. dose plan

wahln commented 2 years ago

The warnings that you see are normal and should not stop matRad from importing RTdose. First, just to make sure that the workflow is correct: After scanning, you need to select the cubes you want to import specifically, and then click import. Now, some other questions/comments:

What modality are you importing? (IMRT, VMAT, protons, ...?)
RTPlan import only works reliably if you have a matching machine file. For photons, it will usually just import anyway, but particle plans is a different thing, as our tabulated depth dose tables do usually not match the energies requested in the DICOM file.

pietemma commented 2 years ago

the modality is SIB - boost VMAT its for photons How do you select the cubes on a standalone?

wahln commented 2 years ago

It should be possible to just click the warnings away, then highlight the dose file in the RT Dose window by click and then click import. With the standalone, I can't really check what's going wrong in detail, as it does not print sophisticated error messages.

pietemma commented 2 years ago

Thanks for the reply, can i send you a sample of the data in a zipped file so that you can check and see what could be the cause of the error, maybe its the data. i cannot access matlab in my location.

wahln commented 2 years ago

If you contact us via the e-mail address stated here https://github.com/e0404 or on our starting page https://e0404.github.io/matRad/ (I don't want to type the e-mail address as I don't know how well the e-mail spam protection within issues works), we can discuss if/how we may take a look at your data. Just make sure you are allowed to share it (best case would be a phantom that has the same import problems).

wahln commented 2 years ago

So there's indeed a stupid bug here in the code here:
https://github.com/e0404/matRad/blob/3d29b765a9a0a02a593890726102a7d9db907af7/dicom/matRad_importDicom.m#L127-L144

We arrive at this loop with an empty pln and then never enter the dose import. If we try to import the plan itself, it also fails due to having archs.

We can check it out further soon, but currently there's quite some other action going on here and it may take a while, especially since you would need a compiled standalone update.

wahln commented 2 years ago

I implemented a fix in #574 . I could build a standalone version including the fix for you if you still need it (on windows?)