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: Fix vineyard compatibility issue #783

Closed luweizheng closed 1 month ago

luweizheng commented 3 months ago

What do these changes do?

Fix vineyard compatibility issue. Note that there is a a workaround here: vineyard return a different class name (metadata), while the data values is correct.

Related issue number

Fixes #xxxx

Check code requirements

luweizheng commented 3 months ago

The vineyard CI error may be related to pandas 2.2.2. See: https://github.com/v6d-io/v6d/issues/1930

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.97%. Comparing base (223940f) to head (359ac32). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #783 +/- ## ========================================== + Coverage 80.70% 80.97% +0.27% ========================================== Files 1070 1070 Lines 80015 80015 Branches 16522 16522 ========================================== + Hits 64575 64796 +221 + Misses 13881 13644 -237 - Partials 1559 1575 +16 ``` | [Flag](https://app.codecov.io/gh/xorbitsai/xorbits/pull/783/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/783/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xorbitsai) | `80.93% <ø> (+0.27%)` | :arrow_up: | 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.