Conversation
|
@PSSF23 what did you do about the MLKNN unit test? |
Nothing. Unrelated. |
|
The unit tests seem to fail. |
Yes. But it's not related to this PR. I made another issue #548 for that. |
It's bad practice to merge PRs without all checks being passed! It will be better to possibly take out that unit test or use @pargma no cover to ignore the unit test. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## staging #547 +/- ##
===========================================
- Coverage 92.36% 88.54% -3.82%
===========================================
Files 7 7
Lines 419 419
===========================================
- Hits 387 371 -16
- Misses 32 48 +16 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Reference issue
Close #546
Close #543
Type of change
Documentation, Website
What does this implement/fix?
Additional information