rapidsai/cudf

[FEA] Clean up `type_to_name` and add `print_type` debug utility

Open

#17,376 创建于 2024年11月19日

在 GitHub 查看
 (5 评论) (1 反应) (0 负责人)C++ (6,000 star) (735 fork)batch import
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_type with type_to_name
  • Replace the type stringify logic in benchmarks/io/nvbench_helpers.hpp with type_to_name
  • Add a host-device print_type utility in cudf_test/print_utilities.cuh using type_to_name for debugging (Note that making it host-device is not mandatory, as type_to_name is currently host-only. However, enabling its use on both host and device, as required by print_type, would involve additional work, such as relocating it to a .cuh header and potentially modifying downstream usage.)

贡献者指南

[FEA] Clean up `type_to_name` and add `print_type` debug utility · rapidsai/cudf#17376 | Good First Issue