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

Destaggering of the point cloud #66

Closed
wants to merge 19 commits into from

Conversation

66lukas66
Copy link

Related Issues & PRs

Issus #61 and #60 Destaggering of a Point cloud

Summary of Changes

Adding the function destagger in ous.ros with a option to destagger or not.

Validation

using of the new function checkofDestagger who check if a point cloud is destaggered

@Samahu Samahu self-requested a review February 24, 2023 00:59
@66lukas66
Copy link
Author

src/os_cloud_nodelet.cpp
The destagger cloud and points would have to be installed there so that a staggered as well as a destaggered cloud can be published.

@66lukas66
Copy link
Author

.github/workflows/docker-image.yml
ouster-sdk

the merge conflicts in this parts a fixt

@Samahu
Copy link
Contributor

Samahu commented Aug 9, 2023

Hi @66lukas66, The PRs #182, #187 and #188 implements the functionality set in this PR (with slight differences: destaggering is performed during copy of the PCL point cloud rather than in a separate step), as such I am closing the PR.

@Samahu Samahu closed this Aug 9, 2023
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.

2 participants