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

Round contour.ContourData to 10 decimal places #81

Merged
merged 3 commits into from
Feb 28, 2023

Conversation

tomaroberts
Copy link
Contributor

@tomaroberts tomaroberts commented Feb 13, 2023

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

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

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

@tomaroberts
Copy link
Contributor Author

@asim-shrestha

Pushed the requested change. Thanks :)

@tomaroberts
Copy link
Contributor Author

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

@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 🙏

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

Successfully merging this pull request may close these issues.

None yet

2 participants