Skip to content
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

[BUG] DICOM import does not import RTdose #558

Open
pietemma opened this issue May 24, 2022 · 7 comments
Open

[BUG] DICOM import does not import RTdose #558

pietemma opened this issue May 24, 2022 · 7 comments
Assignees
Labels

Comments

@pietemma
Copy link

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

@pietemma pietemma added the bug label May 24, 2022
@wahln wahln changed the title [BUG] [BUG] DICOM import does not import RTdose May 24, 2022
@wahln
Copy link
Contributor

wahln commented May 24, 2022

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
Copy link
Author

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

@wahln
Copy link
Contributor

wahln commented May 30, 2022

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
Copy link
Author

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
Copy link
Contributor

wahln commented Jun 20, 2022

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
Copy link
Contributor

wahln commented Jun 23, 2022

So there's indeed a stupid bug here in the code here:

if isfield(files,'rtdose')
% check if files.rtdose contains a path and is labeld as RTDose
% only the first two elements are relevant for loading the rt dose
if ~(cellfun(@isempty,files.rtdose(1,1:2)))
fprintf('loading Dose files \n', structures(i).structName);
% parse plan in order to scale dose cubes to a fraction based dose
if exist('pln','var')
if isfield(pln,'numOfFractions')
resultGUI = matRad_importDicomRTDose(ct, files.rtdose, pln);
end
else
resultGUI = matRad_importDicomRTDose(ct, files.rtdose);
end
if size(resultGUI) == 0
clear resultGUI;
end
end
end

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
Copy link
Contributor

wahln commented Aug 11, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants