Skip to content

Rename sort_and_shuffle to groupby_gpuid_and_shuffle#1392

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.19from
seunghwak:enh_rename_shuffle
Feb 22, 2021
Merged

Rename sort_and_shuffle to groupby_gpuid_and_shuffle#1392
rapids-bot[bot] merged 4 commits intorapidsai:branch-0.19from
seunghwak:enh_rename_shuffle

Conversation

@seunghwak
Copy link
Copy Markdown
Contributor

Rename to better reflect what this function should do than how it is currently implemented (which can change in the future for better performance).

@seunghwak seunghwak requested review from a team as code owners February 9, 2021 19:18
@seunghwak seunghwak added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 9, 2021
@BradReesWork BradReesWork added this to the 0.19 milestone Feb 10, 2021
@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 10, 2021

Codecov Report

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

Impacted file tree graph

@@              Coverage Diff               @@
##             branch-0.19    #1392   +/-   ##
==============================================
  Coverage               ?   60.72%           
==============================================
  Files                  ?       70           
  Lines                  ?     3132           
  Branches               ?        0           
==============================================
  Hits                   ?     1902           
  Misses                 ?     1230           
  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 591b5e7...dff89e1. Read the comment docs.

Copy link
Copy Markdown
Member

@afender afender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@BradReesWork
Copy link
Copy Markdown
Member

@gpucibot merge

@rapids-bot rapids-bot Bot merged commit 369beee into rapidsai:branch-0.19 Feb 22, 2021
@seunghwak seunghwak deleted the enh_rename_shuffle branch June 24, 2021 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants