qurit / rt-utils

A minimal Python library to facilitate the creation and manipulation of DICOM RTStructs.
MIT License
181 stars 56 forks source link

Round contour.ContourData to 10 decimal places #81

Closed tomaroberts closed 1 year ago

tomaroberts commented 1 year ago

See issue #80 for fuller explanation.

This PR rounds contour.ContourData to 10 decimal places to ensure it is <16 bytes length, as per NEMA DICOM standard guidelines.

tomaroberts commented 1 year ago

Hi devs... Just to add, I've implemented and tested this offline with our systems – can confirm that this fixes the RTStructs in our pipeline so they can be successfully opened in Eclipse.

asim-shrestha commented 1 year ago

Hey @tomaroberts, thanks for looking into this! Happy to merge this in once the comment is resolved :)

tomaroberts commented 1 year ago

@asim-shrestha

Pushed the requested change. Thanks :)

tomaroberts commented 1 year ago

@asim-shrestha – Thanks for merging this. When do you expect to do a new release to PyPI? Would be great if I can pip install with the latest changes.

tomaroberts commented 1 year ago

@qurit-frizi

Do you, or someone from the team, know when rt-utils is going to get another release on PyPI? I could really do with an up-to-date PyPI release which includes this merged PR. I've tried packaging rt-utils using the main branch, but can't get it working as needed. Thanks in advance 🙏