Conversation
|
This is a nice addition! Is there anybody reviewing this? Did you also run simple consistency checks with matRad only, i.e., exporing with matRad and reimporting with matRad? |
|
@becker89 wanted to look at it, I guess.. Reimporting in matRad again is the smallest issue, so far it worked with matRad, CERR and MITK. Would a RTPlan export make sense with our generic machines or not? |
|
IMO a DICOM export for particle plans does not make sense. This should be driven by a clear usecase e.g. communication based on HIT basedata with Syngo or Raystation or a likewise scenario at a different institution. Do you have something like this in the pipeline at dkfz? |
|
We thought a little bit of it... I would see a generic export more as a base code for other people to adapt to their machines. |
|
anyway: @becker89 are you going to review this or should I look at it (might take a while)... |
|
I will review the code on Thursday or Friday. |
|
Great. Thanks!
…On Tue, Sep 10, 2019 at 1:33 AM Hans-Peter Wieser ***@***.***> wrote:
I will review the code on Thursday or Friday.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#368?email_source=notifications&email_token=ACNPY7IZUOKI6YNPB5DG6NDQI3MK5A5CNFSM4IPKHB32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6JK4EY#issuecomment-529706515>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACNPY7IAZJIZZZVEMZLPDELQI3MK5ANCNFSM4IPKHB3Q>
.
|
* tested dicom export with Slicer RT (including RT dose) * tested on Mac OS
*support of multiple contours of the same VOI on a single slice
|
Tested the exported DICOM data with Slicer RT. The vt, cst and the dose cube was correctly exported from matRad and could be successfully imported to Slicer RT. |
| end | ||
|
|
||
|
|
||
| resultName = strcat(doseTypeHelper,'_',num2str(doseInstanceHelper),beamSuffix); |
There was a problem hiding this comment.
I think the additional underscore mess up resultGUI in the end. You will find a 'resultGUI.physicalDose_' after import in your workspace. If you now try to recalc, you get an 'reference to a non existing field 'w_' ' error dialog.
In my case this error occurs because of a missing InstanceNumber.
To keep the functionality one could edit '_',num2str(doseInstanceHelper) like beamSuffix in line 84-86 and add this to line 51 or we should accordingly edit the string concatenation/creation of resultGUI.w.
Dicom Exporter
4 participants
Rudimentary dicom export for CT, RTStruct & RTDose. Tested by reimporting exported TG119 + dose into CERR & MITK. Also adds differentiation between full / fraction / beam dose to the import.
May add RTPlan in the future if it makes sense.