Aditya Panchal

Great, I'll start looking into these shortly. On Tue, Nov 17, 2015 at 2:03 AM, richardmuellerx [email protected] wrote: > Ok, the sample DICOM files are uploaded. > > — >...

I will look into this shortly. I know there are some conditions that it doesn't work with, but are you able to provide some example (anonymized) data to test with?...

Thanks for the pull request, will take a look soon. I am currently working on dependency updates so that I can get dicompyler running on up-to-date software libraries.

The DVH data extraction has been refactored into the separate `dicompyler-core` project. I don't think the out of bound issue should be present as that code is reworked completely. I...

Consider adding UID anonymization per the thread here: https://groups.google.com/d/topic/dicompyler/acpJzICYLPI/discussion Need to make sure that the UID is valid and less than 64 characters. Also if anonymizing a DICOM RT set,...

I actually had a similar idea when I set out to make dicompyler/dicompyler-core which would have modules for DICOM RT. So far only one module (DVH) is fully complete as...

Did you try checking the volume size with interpolation in `get_dvh`? If the structure is small relative to the RT Dose grid size, the voxellization process does not work well....

Will need to do some more investigation. A quick workaround is to use the volume calculated from `CalculateStructureVolume` and manually set the volume on the `dvh` instance.

> This can be solved by using: `get_dvh(... calculate_full_volume=True ...)` @SergRuset is correct. Does this resolve all the issues? I suppose I need to check if the structure is outside...

What a coincidence, I was just thinking about this last night. I'll definitely take a look at your code and see if we can roll it in. Another question, what...