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

ENH: Add support for composite ROIs #20

Merged
merged 2 commits into from
May 27, 2020

Conversation

scottclowe
Copy link
Collaborator

@scottclowe scottclowe commented May 25, 2020

I based this on these sources:

The composite type used to be returned a rectangle containing only the bounding box around the composite ROI. We still return this bounding box data, but now also the paths contained within the composite ROI. The 'paths' field of the composite ROI contains a list of paths describing the ROI. Each path is made up of a series of nodes, encoded as a pair of values in a tuple. We support the ability to decode PathIterator segments which are quadratic or cubic, but it doesn't appear that ImageJ ever saves such paths (it appears to be impossible to save a composite ROIs with sub-pixel resolution and hence paths are always linear).

Closes #9.

@hadim
Copy link
Owner

hadim commented May 27, 2020

Thank you for this! Looks like you need to rebase.

@hadim hadim mentioned this pull request May 27, 2020
@scottclowe
Copy link
Collaborator Author

Ah, I was expecting this. Will rebase onto master now.

@hadim hadim merged commit 881d65f into hadim:master May 27, 2020
@scottclowe scottclowe deleted the enh_composite-rois branch May 27, 2020 19:40
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.

Cannot read in composite ROIs
2 participants