xorbitsai / xorbits

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

Fix reduction agg with numpy udf #742

Closed luweizheng closed 8 months ago

luweizheng commented 8 months ago

What do these changes do?

Similar issue like https://github.com/xorbitsai/xorbits/pull/737

Check code requirements

codecov[bot] commented 8 months ago

Codecov Report

Merging #742 (f496ca1) into main (b1d828f) will decrease coverage by 5.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #742      +/-   ##
==========================================
- Coverage   93.34%   88.30%   -5.04%     
==========================================
  Files        1026     1026              
  Lines       79589    79589              
  Branches    16497    16497              
==========================================
- Hits        74290    70281    -4009     
- Misses       3616     7479    +3863     
- Partials     1683     1829     +146     
Flag Coverage Δ
unittests 88.20% <100.00%> (-5.04%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
python/xorbits/_mars/dataframe/reduction/max.py 100.00% <100.00%> (ø)
python/xorbits/_mars/dataframe/reduction/mean.py 100.00% <100.00%> (ø)
python/xorbits/_mars/dataframe/reduction/min.py 100.00% <100.00%> (ø)

... and 144 files with indirect coverage changes