PointCloudLibrary/pcl

Inconsistent handling of default seed in NormalGenerator & UniformGenerator

Open

#2,985 创建于 2019年4月9日

在 GitHub 查看
 (3 评论) (1 反应) (0 负责人)C++ (4,506 fork)batch import
effort: lowgood first issuekind: todoneeds: feedback

仓库指标

Star
 (9,023 star)
PR 合并指标
 (平均合并 8天 7小时) (30 天内合并 3 个 PR)

描述

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.

贡献者指南

研究方向
检查 PR #2956 中的讨论以及 random.h（第163-186行）和 random.hpp（第49行）中链接的代码行。问题在于默认种子值不一致（ 1 与 1）以及一个有符号/无符号比较警告。修复方法包括选择一个一致的默认值（可能为 0）并调整比较以使用相同类型。同时考虑种子值 1 是否应具有特殊含义。
技术栈
cpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Git
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65