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

Support adding ROIs from polygon data #79

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

zeakey
Copy link

@zeakey zeakey commented Feb 9, 2023

I've implemented the functionality of adding ROIs from polygons (mentioned in #77 ). I tried to keep the modifications as less as possible and keep it backward-compatitable.

  • added a Polygon2D class to handle polygon data
  • Changed ROIData init function to accepts polygon data
  • update the logic in RTStruct.add_roi to selectively construct ROI from mask or polygon
  • add validate_polygon for RTStruct

Potential bugs on existing codes:

Since I changed the init function of ROIData, existing code with positional args may cause errors.

* added a `Polygon2D` class to handle polygon data
* Changed ROIData __init__ function to accepts polygon data
* update the logic in `RTStruct.add_roi` to selectively construct ROI from mask or polygon
* add `validate_polygon` for `RTStruct`
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.

1 participant