rapidsai/cudf

[FEA] Improve readability of thread id based branching

Aperta

#6241 aperta il 15 set 2020

Vedi su GitHubGuida contributor
 (6 commenti) (2 reazioni) (0 assegnatari)C++ (735 fork)batch import
cuIOfeature requestgood first issuelibcudf

Metriche repository

Star
 (6000 stelle)
Metriche merge PR
 (Merge medio 17g 21h) (230 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Sostituisci i controlli grezzi dell'ID del thread (ad esempio `!t`, `lane id == 0`) con nomi descrittivi e utilizza gruppi cooperativi dove appropriato per una migliore leggibilità.
Tech stack
cpp
Dominio
backenddata
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++CUDA
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40