rapidsai/cudf
在 GitHub 查看[FEA] Clean up `type_to_name` and add `print_type` debug utility
Open
#17,376 建立於 2024年11月19日
feature requestgood first issuelibcudf
描述
Is your feature request related to a problem? Please describe.
PR #17194 introduces a type_id to string utility located within the benchmark code.
@ttnghia identified that this could be a shared utility for both benchmarks and tests, as it could also be used to create a print_type utility for debugging purposes (see: https://github.com/rapidsai/cudf/pull/17194/files#r1849064965).
@davidwendt highlighted that this feature is redundant and performs a similar function to type_to_name.
Describe the solution you'd like
- Once #17194 is merged, replace
stringify_typewithtype_to_name - Replace the type stringify logic in
benchmarks/io/nvbench_helpers.hppwithtype_to_name - Add a host-device
print_typeutility incudf_test/print_utilities.cuhusingtype_to_namefor debugging (Note that making it host-device is not mandatory, astype_to_nameis currently host-only. However, enabling its use on both host and device, as required byprint_type, would involve additional work, such as relocating it to a.cuhheader and potentially modifying downstream usage.)