xorbitsai / xorbits

Scalable Python DS & ML, in an API compatible & lightning fast way.
https://xorbits.readthedocs.io
Apache License 2.0
1.11k stars 67 forks source link

ENH: Remove `sum_over_features` parameter from `manhattan_distances` with sklearn > 1.4 #779

Closed Dawnfz-Lenfeng closed 3 months ago

Dawnfz-Lenfeng commented 3 months ago

What do these changes do?

Fixes #xxxx

Check code requirements

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 68.84%. Comparing base (79c3bc8) to head (bc69412).

Files Patch % Lines
.../xorbits/_mars/learn/metrics/pairwise/manhattan.py 0.00% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #779 +/- ## =========================================== - Coverage 84.60% 68.84% -15.77% =========================================== Files 1059 1059 Lines 79804 79731 -73 Branches 16497 12139 -4358 =========================================== - Hits 67521 54892 -12629 - Misses 10707 23329 +12622 + Partials 1576 1510 -66 ``` | [Flag](https://app.codecov.io/gh/xorbitsai/xorbits/pull/779/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xorbitsai) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/xorbitsai/xorbits/pull/779/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xorbitsai) | `68.83% <0.00%> (-15.71%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xorbitsai#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.