PointCloudLibrary / PointCloudLibrary/pcl

Transform the `sensor_orientation_` and `sensor_origin_` in `transformPointCloud`

Open
#4,164 9 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

effort: low kind: proposal kind: todo module: common
Dominant language
C++
Stars
11.1k
Forks
4.7k
Avg merge
4d 10h
Merged PRs (30d)
6

Description

I think the sensor acquisition pose is so important that it can tell where the view point is when the data is acquired and how the objects could be occluded. However, I found this information is lost after calling pcl::transformPointCloud, which is just copying the previous pose. I am curious about the reasons behind this code and would like to know the possibility of always keeping this information with the point cloud data.

https://github.com/PointCloudLibrary/pcl/blob/0b63c9e8dd63e640998ab587ae9c642eccf3b2c9/common/include/pcl/common/impl/transforms.hpp#L237-L238

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with common/include/pcl/common/impl/transforms.hpp at the linked transformPointCloud code and trace how sensor_orientation_ and sensor_origin_ are handled. Clarify the expected pose semantics for a transformed point cloud, then verify the agreed behavior with relevant transformPointCloud coverage; done means the sensor metadata is preserved or transformed consistently rather than copied unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-vision
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.