-
Notifications
You must be signed in to change notification settings - Fork 129
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
Add SAMBboxToInstanceMask transform #1133
Add SAMBboxToInstanceMask transform #1133
Conversation
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
6815c64
to
1376999
Compare
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #1133 +/- ##
===========================================
+ Coverage 80.16% 80.22% +0.05%
===========================================
Files 260 264 +4
Lines 29370 29494 +124
Branches 5757 5779 +22
===========================================
+ Hits 23545 23661 +116
- Misses 4484 4487 +3
- Partials 1341 1346 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
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.
LGTM
Summary
How to test
Added unit tests for this change.
Checklist
License
Feel free to contact the maintainers if that's a concern.