Skip to content

add support code for polygon workflows - #1419

Merged
bw4sz merged 1 commit into
weecology:mainfrom
jveitchmichaelis:polygon-glue
Aug 30, 2026
Merged

add support code for polygon workflows#1419
bw4sz merged 1 commit into
weecology:mainfrom
jveitchmichaelis:polygon-glue

Conversation

@jveitchmichaelis

Copy link
Copy Markdown
Collaborator

Description

This PR adds a number of support features for polygon modeling. While the exact models are under discussion, the additions in this PR are generic:

  • Visualization
  • Geometry format detection
  • Metric support
  • COCO RLE mask decoding
  • Mask <> Polygon conversion
  • Polygon NMS

AI-Assisted Development

Had Claude pluck this out from another branch and then make significant edits to the language + style, again.

AI tools used (if applicable):

Claude Opus 5.

@jveitchmichaelis
jveitchmichaelis force-pushed the polygon-glue branch 3 times, most recently from 1d57eeb to e42d719 Compare August 5, 2026 01:23
@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.38053% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.28%. Comparing base (2b26da4) to head (4b7faa4).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/deepforest/predict.py 68.00% 8 Missing ⚠️
src/deepforest/utilities.py 95.38% 3 Missing ⚠️
src/deepforest/visualize.py 94.11% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1419      +/-   ##
==========================================
+ Coverage   86.67%   87.28%   +0.60%     
==========================================
  Files          28       28              
  Lines        4069     4129      +60     
==========================================
+ Hits         3527     3604      +77     
+ Misses        542      525      -17     
Flag Coverage Δ
unittests 87.28% <89.38%> (+0.60%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jveitchmichaelis

Copy link
Copy Markdown
Collaborator Author

Added some tests. Will put a separate PR in to add unit tests for predict.py as coverage is a bit scattered there.

@jveitchmichaelis
jveitchmichaelis marked this pull request as ready for review August 6, 2026 16:59
@jveitchmichaelis

Copy link
Copy Markdown
Collaborator Author

Actions is currently degraded, so waiting to re-run

@jveitchmichaelis
jveitchmichaelis requested review from bw4sz and henrykironde and removed request for bw4sz August 27, 2026 19:06

@bw4sz bw4sz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great, this looks ready for main.

@bw4sz
bw4sz merged commit be14411 into weecology:main Aug 30, 2026
7 of 13 checks passed
@jveitchmichaelis
jveitchmichaelis deleted the polygon-glue branch August 30, 2026 22:26
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.

2 participants