rapidsai/cudf

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

Fechada

#17.376 aberto em 19 de nov. de 2024

Ver no GitHubGuia do colaborador
 (5 comentários) (1 reação) (0 responsável)C++ (735 forks)batch import
feature requestgood first issuelibcudf

Métricas do repositório

Stars
 (6.000 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.)

Guia do colaborador

Direção de pesquisa
Localize `stringify type` no código de benchmarks e substitua por `type to name` do cudf. Em seguida, adicione um utilitário `print type` host dispositivo em `cudf test/print utilities.cuh` usando `type to name`. Certifique se de lidar com a possível realocação de `type to name` para um cabeçalho .cuh para uso no dispositivo.
Pilha de tecnologia
c
Domain
backenddata
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++CUDA
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30