Fix/ImgDetectionFilter segmentation mask reindexing#306
Conversation
…ltered detection list indices
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthrough
ChangesDetection filtering and mask handling
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #306 +/- ##
==========================================
+ Coverage 48.18% 48.21% +0.03%
==========================================
Files 99 99
Lines 6438 6442 +4
==========================================
+ Hits 3102 3106 +4
Misses 3336 3336 ☔ View full report in Codecov by Harness. |
Purpose
Reindex segmentation masks in ImgDetectionsFilter to match the new filtered detection list indices.
Specification
None / not applicable
Dependencies & Potential Impact
None / not applicable
Deployment Plan
None / not applicable
Testing & Validation
Added unit tests for this modified behavior. Validated all oak-examples with ImgDetectionsFilter still work as before.
AI Usage
Assisted-by: Pi Coding Agent:GPT-5.5
Submitted code was reviewed by a human: YES
The author is taking the responsibility for the contribution: YES
Summary by CodeRabbit
Bug Fixes
Tests