rapidsai/cudf

[FEA] Improve readability of thread id based branching

Open

#6.241 aberto em 15 de set. de 2020

Ver no GitHub
 (6 comments) (2 reactions) (0 assignees)C++ (735 forks)batch import
cuIOfeature requestgood first issuelibcudf

Métricas do repositório

Stars
 (6.000 stars)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (230 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Substitua as verificações brutas de ID de thread (por exemplo, `!t`, `lane id == 0`) por nomes descritivos e use grupos cooperativos quando apropriado para melhor legibilidade.
Pilha de tecnologia
cpp
Domain
backenddata
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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.
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.
40