Skip to content

Migrate T-digest test helpers to memory_resources - #23608

Draft
bdice wants to merge 1 commit into
NVIDIA:mainfrom
bdice:test-mr-tdigest-main
Draft

Migrate T-digest test helpers to memory_resources#23608
bdice wants to merge 1 commit into
NVIDIA:mainfrom
bdice:test-mr-tdigest-main

Conversation

@bdice

@bdice bdice commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Description

Update the installed T-digest generators, expected-column builders, validators, and shared groupby/reduction drivers to accept cudf::memory_resources. Returned T-digest columns use the output resource; input construction, intermediate results, and validation scratch use the temporary resource.

This allows T-digest tests to control their setup and validation allocations without falling back to the current device resource.

This is a non-breaking change for existing callers of these test helpers.

Depends on #23581.

Part of #20780

Checklist

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

@copy-pr-bot

copy-pr-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the libcudf Affects libcudf (C++/CUDA) code. label Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libcudf Affects libcudf (C++/CUDA) code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant