-
Notifications
You must be signed in to change notification settings - Fork 128
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
Fixing KNNCodecServiceTests as reported in the gh issue:[#1593] #1601
Conversation
00eb4f6
to
d0b0510
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1601 +/- ##
============================================
- Coverage 84.92% 84.87% -0.06%
+ Complexity 1375 1374 -1
============================================
Files 172 172
Lines 5605 5605
Branches 553 553
============================================
- Hits 4760 4757 -3
- Misses 612 614 +2
- Partials 233 234 +1 ☔ 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! Thanks
…roject#1593] Signed-off-by: Navneet Verma <[email protected]>
5f0ba36
d0b0510
to
5f0ba36
Compare
Signed-off-by: Navneet Verma <[email protected]> (cherry picked from commit 5de10fc)
… (#1663) Signed-off-by: Navneet Verma <[email protected]> (cherry picked from commit 5de10fc) Co-authored-by: Navneet Verma <[email protected]>
Description
Fixing KNNCodecServiceTests as reported in the gh issue:[#1593]
Currently the issue is only in main branch.
Issues Resolved
#1593
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.