PointCloudLibrary/pcl

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

Offen

#3.781 geöffnet am 23.03.2020

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (4.506 Forks)batch import
effort: mediumgood first issuekind: todo

Repository-Metriken

Stars
 (9.023 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8T 7h) (3 gemergte PRs in 30 T)

Beschreibung

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)

Contributor Guide

Research-Richtung
Durchsuchen Sie die gesamte Codebasis nach der Verwendung von `rand` und ersetzen Sie diese durch C++11 `<random>` Hilfsmittel wie `std::default random engine` und `std::uniform int distribution`. Stellen Sie Konsistenz zwischen Header und Quelldateien sicher.
Tech Stack
cpp
Domain
computer graphics
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++C++11 random library
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60