PointCloudLibrary/pcl

Inconsistent handling of default seed in NormalGenerator & UniformGenerator

Open

#2,985 opened on 2019年4月9日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)C++ (4,506 forks)batch import
effort: lowgood first issuekind: todoneeds: feedback

Repository metrics

Stars
 (9,023 stars)
PR merge metrics
 (平均マージ 8d 7h) (30d で 3 merged PRs)

説明

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（L163-186）および random.hpp（L49）のリンクされた行を検討してください。問題は、デフォルトシード値の不一致（ 1 対 1）と、符号付き/符号なし比較の警告です。修正には、一貫したデフォルト値（おそらく 0）の選択と、比較を同じ型を使用するように調整することが含まれます。また、シード値 1 に特別な意味を持たせるべきかどうかも検討してください。
技術スタック
cpp
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++Git
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65