PointCloudLibrary/pcl

Inconsistent handling of default seed in NormalGenerator & UniformGenerator

Aberta

#2.985 aberto em 9 de abr. de 2019

Ver no GitHubGuia do colaborador
 (3 comentários) (1 reação) (0 responsável)C++ (4.506 forks)batch import
effort: lowgood first issuekind: todoneeds: feedback

Métricas do repositório

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

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.

Guia do colaborador

Direção de pesquisa
Examine a discussão no PR #2956 e as linhas vinculadas em random.h (L163-186) e random.hpp (L49). O problema são valores de semente padrão inconsistentes ( 1 vs 1) e um aviso de comparação signed/unsigned. A correção envolve escolher um valor padrão consistente (provavelmente 0) e ajustar a comparação para usar o mesmo tipo. Considere também se a semente 1 deve ter um significado especial.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65