rapidsai/cudf

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

Open

#17.376 geöffnet am 19. Nov. 2024

Auf GitHub ansehen
 (5 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (735 Forks)batch import
feature requestgood first issuelibcudf

Repository-Metriken

Stars
 (6.000 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 17T 21h) (230 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Finden Sie `stringify type` im Benchmark Code und ersetzen Sie es durch `type to name` aus cudf. Fügen Sie dann ein Host Device Dienstprogramm `print type` in `cudf test/print utilities.cuh` hinzu, das `type to name` verwendet. Stellen Sie sicher, dass Sie die mögliche Verschiebung von `type to name` in einen .cuh Header für die Geräteverwendung berücksichtigen.
Tech Stack
c
Domain
backenddata
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++CUDA
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30