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 ReportBase: 92.36% // Head: 88.54% // Decreases project coverage by
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
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Reference issue
Close #546
Close #543
Type of change
Documentation, Website
What does this implement/fix?
Additional information