Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added separate helpers for moving buffers to either cudf column and series objects #1915

Merged

Conversation

rlratzel
Copy link
Contributor

This is a follow-up PR to address this feedback from #1901

This adds a separate cython helper for moving a buffer to a column to allow for a slightly more efficient call where column objects can be used.

…since columns are often acceptable and the extra conversion to series can be skipped.
@rlratzel rlratzel added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 28, 2021
@rlratzel rlratzel added this to the 21.12 milestone Oct 28, 2021
@rlratzel rlratzel self-assigned this Oct 28, 2021
@rlratzel rlratzel requested a review from a team as a code owner October 28, 2021 23:10
@codecov-commenter
Copy link

Codecov Report

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

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.12    #1915   +/-   ##
===============================================
  Coverage                ?   69.53%           
===============================================
  Files                   ?      143           
  Lines                   ?     8626           
  Branches                ?        0           
===============================================
  Hits                    ?     5998           
  Misses                  ?     2628           
  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 731065f...cb65876. Read the comment docs.

@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit bfa7f92 into rapidsai:branch-21.12 Nov 3, 2021
@rlratzel rlratzel deleted the branch-21.12-addcythoncolumnhelper branch June 17, 2022 00:32
# for free to join this conversation on GitHub. Already have an account? # 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