Skip to content

Replace weight by safe_sizeof#584

Merged
rapids-bot[bot] merged 3 commits intorapidsai:branch-0.20from
pentschev:fix-weight
Apr 21, 2021
Merged

Replace weight by safe_sizeof#584
rapids-bot[bot] merged 3 commits intorapidsai:branch-0.20from
pentschev:fix-weight

Conversation

@pentschev
Copy link
Member

Fix for changes in dask/distributed#4651

@pentschev pentschev requested a review from a team as a code owner April 20, 2021 15:24
@github-actions github-actions bot added the python python code needed label Apr 20, 2021
@pentschev pentschev added 3 - Ready for Review Ready for review by team bug Something isn't working non-breaking Non-breaking change labels Apr 20, 2021
@pentschev
Copy link
Member Author

rerun tests

@pentschev
Copy link
Member Author

It seems that there are other issues in Distributed: dask/distributed@d5fc324#r49759517 . I'll retarget this to 0.20 and pin Distributed 2021.04.0 in 0.19 just as dask-cudf does.

@pentschev pentschev changed the base branch from branch-0.19 to branch-0.20 April 20, 2021 16:18
@jrbourbeau
Copy link
Member

Thanks for tracking these issues down @pentschev. xref-ing dask/distributed#4723 which adds weight back in to avoid any issues with distributed releasing before dask-cuda

@pentschev
Copy link
Member Author

rerun tests

@codecov-commenter
Copy link

codecov-commenter commented Apr 21, 2021

Codecov Report

❗ No coverage uploaded for pull request base (branch-0.20@81cd211). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             branch-0.20     #584   +/-   ##
==============================================
  Coverage               ?   60.41%           
==============================================
  Files                  ?       22           
  Lines                  ?     2650           
  Branches               ?        0           
==============================================
  Hits                   ?     1601           
  Misses                 ?     1049           
  Partials               ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 81cd211...34f0fbf. Read the comment docs.

@pentschev
Copy link
Member Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d3c723e into rapidsai:branch-0.20 Apr 21, 2021
@pentschev pentschev deleted the fix-weight branch April 23, 2021 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team bug Something isn't working non-breaking Non-breaking change python python code needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants