-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/semseg polygon dep #592
Conversation
height=[img.shape[-1] for img in self.gold_masks], | ||
width=[img.shape[-2] for img in self.gold_masks], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is duplicated on 210 below, lets just call it once and reuse var
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks nice! can you create a new run called polygon-dep
to test before you merge?
i can take a look at the processed DF and make sure it saved right
Co-authored-by: Elliott <elliott@rungalileo.io>
Codecov Report
@@ Coverage Diff @@
## main #592 +/- ##
==========================================
- Coverage 84.88% 84.71% -0.17%
==========================================
Files 161 161
Lines 12598 12624 +26
==========================================
+ Hits 10694 10695 +1
- Misses 1904 1929 +25
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Adds polygon dep as an attribute and compute dep for each polygon.
Also adds a helper function to polygon_data to make it more readable