Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Laser origins #1395

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Laser origins #1395

wants to merge 9 commits into from

Conversation

demmerichs
Copy link

This PR only adds 2 more commits on top of PR #1394 and introduces pointwise laser origins in the dataset loading and processing routine, enabled through the dataset config key USE_ORIGINS=True.
Laser origins can be interesting in order to track the laser rays shoot by a LiDAR scanner, in order to deduce empty space. E.g. MAELi makes use of this visibility concept.

I tested all changes and produced the following image from the loaded 'points' and 'origins' in the data_dict.
Screenshot from 2023-06-23 18-19-48
This images shows four consecutive point clouds loaded from the Waymo Open Dataset, the different shading of blue shows the different scans (dark newest to light oldest). The reddish colors show the position of the 5 LiDAR sensors and the outward traveling beams, corrected for the movement of the ego vehicle, hence different origins for each scan (yellow newest to red oldest). You can see how the Top LiDAR is 360deg while the lower LiDARs only are less than 180deg.
This demonstrates that the origin date is correctly corresponding to the point data.

I am happy to incorporate any feedback if this PR is deemed to be merged into the base repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant