NVIDIA / NVIDIA/cudf

[FEA] Proposal to replace `cudf::test` namespace with `cudf_test` and introduce a `cudf_benchmark` namespace

Open
#20,764 0 comments 1 reaction 0 assignees View on GitHub
feature request libcudf non-breaking
Dominant language
C++
Stars
9.8k
Forks
1.1k
Avg merge
3d 6m
Merged PRs (30d)
278

Description

There can be confusion in the test and benchmark source on where APIs are implemented and how they are expected to behave since the `cudf::test` namespace is technically also in the `cudf` namespace.
Likewise for the libcudf benchmarks, there is no namespace and so it makes it difficult to know what are local utilities vs benchmark common utilities.

I'd like to introduce the `cudf_test` namespace in place of the current `cudf::test` namespace.
And also introduce the `cudf_benchmark` namespace for benchmark common utilities.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.