PointCloudLibrary/pcl

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

Open

#3781 aperta il 23 mar 2020

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
effort: mediumgood first issuekind: todo

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Cerca in tutto il codebase l'uso di `rand` e sostituiscilo con le utility C++11 `<random>` come `std::default random engine` e `std::uniform int distribution`. Assicura coerenza tra header e file sorgente.
Tech stack
cpp
Dominio
computer graphics
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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++C++11 random library
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60