Skip to content

Add alloc_size member function to cudf::column and cudf::table #18639

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 7, 2025

Conversation

davidwendt
Copy link
Contributor

Description

Adds a new member function to the cudf::table and cudf::column classes to report the device memory allocation size of the internal device-buffers of itself and its children.
The value will including padding and may not be computable from the logical column type and the number of elements and null-count.

Closes #18462

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@davidwendt davidwendt added feature request New feature or request 3 - Ready for Review Ready for review by team libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change labels May 2, 2025
@davidwendt davidwendt self-assigned this May 2, 2025
@davidwendt davidwendt requested a review from a team as a code owner May 2, 2025 15:40
@davidwendt davidwendt requested review from karthikeyann and vuule May 2, 2025 15:40
@GregoryKimball GregoryKimball moved this to Burndown in libcudf May 4, 2025
@davidwendt davidwendt requested a review from bdice May 6, 2025 11:41
@davidwendt
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 2df89e9 into rapidsai:branch-25.06 May 7, 2025
109 of 110 checks passed
@davidwendt davidwendt deleted the col-alloc-size branch May 7, 2025 13:05
@GregoryKimball GregoryKimball moved this from Burndown to Landed in libcudf May 7, 2025
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 feature request New feature or request libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change
Projects
Status: Landed
Development

Successfully merging this pull request may close these issues.

[FEA] Add a method to cudf::table and cudf::column to get its size in bytes without kernel launches or d -> h memcpy
4 participants