PointCloudLibrary/pcl

Inconsistent handling of default seed in NormalGenerator & UniformGenerator

Ouverte

#2 985 ouverte le 9 avr. 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (1 réaction) (0 personne assignée)C++ (4 506 forks)batch import
effort: lowgood first issuekind: todoneeds: feedback

Métriques du dépôt

Stars
 (9 023 étoiles)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

Description

As already discussed here default seed in random.h is not handled consistently.

https://github.com/PointCloudLibrary/pcl/blob/b4c785bf32b392b4570201e8fe9f97395bd98ba1/common/include/pcl/common/random.h#L163-L186

One constructor of NormalGenerator has as default seed -1 during the other one (via Parameters) has 1 as default seed.

Furthermore you get currently following warning:

common/include/pcl/common/impl/random.hpp:49:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   if(parameters_.seed != -1)

So we should discuss how to adjust code there.

Guide contributeur

Direction de recherche
Examinez la discussion dans la PR #2956 et les lignes liées dans random.h (L163-186) et random.hpp (L49). Le problème concerne des valeurs de graine par défaut incohérentes ( 1 vs 1) et un avertissement de comparaison signé/non signé. La correction implique de choisir une valeur par défaut cohérente (probablement 0) et d'ajuster la comparaison pour utiliser le même type. Considérez également si la graine 1 devrait avoir une signification spéciale.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65