PointCloudLibrary/pcl

Inconsistent handling of default seed in NormalGenerator & UniformGenerator

Open

#2985 aperta il 9 apr 2019

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)C++ (4506 fork)batch import
effort: lowgood first issuekind: todoneeds: feedback

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la discussione nella PR #2956 e le righe collegate in random.h (L163-186) e random.hpp (L49). Il problema riguarda valori di seed predefiniti inconsistenti ( 1 vs 1) e un avviso di confronto con segno/senza segno. La correzione implica scegliere un valore predefinito coerente (probabilmente 0) e adattare il confronto per utilizzare lo stesso tipo. Considera anche se il seed 1 dovrebbe avere un significato speciale.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65