PointCloudLibrary/pcl

[PCL] Use C++11 random generators, not `rand`

Open

#3.781 aberto em 23 de mar. de 2020

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
effort: mediumgood first issuekind: todo

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

Description

Context

rand has several issues, which C++11 random generators don't. New PR often see the rand usage and try to emulate the wrong behavior.

Expected behavior

Remove all rand so when people search for how PCL did its random num generation, they get good quality results

Additional context

Usage of old-fashioned rand discovered by codacity (or maybe try grep)

Guia do colaborador

Direção de pesquisa
Pesquise em toda a base de código o uso de `rand` e substitua pelos utilitários C++11 `<random>` como `std::default random engine` e `std::uniform int distribution`. Garanta consistência entre cabeçalhos e arquivos de origem.
Pilha de tecnologia
cpp
Domain
computer graphics
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
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++C++11 random library
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60