rapidsai/cudf

[FEA] Improve readability of thread id based branching

Ouverte

#6 241 ouverte le 15 sept. 2020

Voir sur GitHubGuide contributeur
 (6 commentaires) (2 réactions) (0 personne assignée)C++ (735 forks)batch import
cuIOfeature requestgood first issuelibcudf

Métriques du dépôt

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

Description

Improve readability of thread id based branches by giving them more descriptive names.

e.g.

if (!t) // is actually a t == 0

and

https://github.com/rapidsai/cudf/blob/57ef76927373d7260b6a0eda781e59a4c563d36e/cpp/src/io/statistics/column_stats.cu#L285 Is actually a lane_id == 0 As demonstrated in https://github.com/rapidsai/cudf/issues/6241#issuecomment-693125331, prefer cooperative groups for this.

and

https://github.com/rapidsai/cudf/blob/85cd56dfb3449140f18c7cef3a3be01ac976fd14/cpp/src/io/parquet/page_enc.cu#L1256 is actually t < 32 lane_id == 31. (I think this might be an oversight, ignore as it might be fixed in #6238 ).

Guide contributeur

Direction de recherche
Remplacez les vérifications brutes d'ID de thread (par exemple `!t`, `lane id == 0`) par des noms descriptifs et utilisez des groupes coopératifs le cas échéant pour une meilleure lisibilité.
Stack technique
cpp
Domaine
backenddata
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
Claire
Prérequis
C++CUDA
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40