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

fix: createGiottoPolygon() further fixes and documentation #177

Merged
merged 6 commits into from
Mar 1, 2024

Conversation

jiajic
Copy link
Member

@jiajic jiajic commented Mar 1, 2024

  • added tests for creation from mask file
  • added runnable examples for data.frame-like, mask image, vector file
  • fix dispatch for character type inputs
  • add internal flip function for terra SpatVectors that has functional parity with the one GiottoClass defines for giottoPolygon
  • add new toy mask and vector files for testing & examples with createGiottoPolygon
  • fix poly_ID setting for mask inputs

jiajic added 6 commits March 1, 2024 03:16
terra has a flip function, but this internal adds the ability to flip over arbitrary x and y values in the same manner that has been implemented for `giottoPolygon` and `giottoPoints`
Also updates to documentation and examples
- also specifically show the needed columns for `data.frame-like` inputs
@jiajic jiajic merged commit 86968f9 into drieslab:dev Mar 1, 2024
1 check passed
@jiajic jiajic deleted the patch_maskpolys branch March 1, 2024 17:30
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lintr found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

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