rapidsai/cudf

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

Open

#17 376 ouverte le 19 nov. 2024

Voir sur GitHub
 (5 commentaires) (1 réaction) (0 assignés)C++ (735 forks)batch import
feature requestgood first issuelibcudf

Métriques du dépôt

Stars
 (6 000 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (230 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Localisez `stringify type` dans le code des benchmarks et remplacez le par `type to name` de cudf. Ajoutez ensuite un utilitaire `print type` hôte périphérique dans `cudf test/print utilities.cuh` en utilisant `type to name`. Assurez vous de gérer le déplacement potentiel de `type to name` vers un en tête .cuh pour une utilisation sur le périphérique.
Stack technique
c
Domaine
backenddata
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++CUDA
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30