PointCloudLibrary/pcl

[features] Fix ShapeContext3DEstimation

已关闭

#6,050 创建于 2024年5月22日

 (6 条评论) (0 个反应) (0 位负责人)C++ (4,506 个派生)batch import
good first issuehelp wantedkind: bugmodule: features

仓库指标

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

描述

Currently, the x-axis (and consequently also the y-axis) are chosen randomly, which makes this feature useless without further processing. The paper by Frome et al says in section 2.1:

We have a degree of freedom in the azimuth direction that we must remove in order to compare shape contexts calculated in different coordinate systems. To account for this, we choose some direction to be Φ0 in an initial shape context, and then rotate the shape context about its north pole into L positions, such that each Φl division is located at the original 0 position in one of the rotations. For descriptor data sets derived from our reference scans, L rotations for each basis point are included, whereas in the query data sets, we include only one position per basis point

It seems like the PCL implementation is missing this additional step. There is a so far unimplemented function, seemingly for this purpose: https://github.com/PointCloudLibrary/pcl/blob/master/features/include/pcl/features/3dsc.h#L224 The task is to implement this function and call it in computeFeature, after computePoint.

When the implementation is fixed, the test in test_shot_estimation.cpp probably has to be updated. Ideally, a new test should be added where this feature is computed twice on the same cloud, and the outputs should be very similar.

贡献者指南