Skip to content

[filters] FrustumCulling: allowing infinite far plane distance#5433

Merged
mvieth merged 6 commits intoPointCloudLibrary:masterfrom
keineahnung2345:frustum_infinite_far_plane
Sep 19, 2022
Merged

[filters] FrustumCulling: allowing infinite far plane distance#5433
mvieth merged 6 commits intoPointCloudLibrary:masterfrom
keineahnung2345:frustum_infinite_far_plane

Conversation

@keineahnung2345
Copy link
Copy Markdown
Contributor

@keineahnung2345 keineahnung2345 commented Sep 16, 2022

  1. check for np_dist and fp_dist
  2. allow infinite fp_dist

Ref: #5423

top, bottom, right, left planes are dependent on far plane's corners, so when fp_dist_ is max value, it should be first set to a valid value and then set back.

Comment thread filters/include/pcl/filters/frustum_culling.h Outdated
Comment thread filters/include/pcl/filters/frustum_culling.h Outdated
Comment thread filters/include/pcl/filters/impl/frustum_culling.hpp Outdated
@mvieth mvieth added changelog: new feature Meta-information for changelog generation module: filters labels Sep 17, 2022
keineahnung2345 and others added 4 commits September 19, 2022 09:32
Co-authored-by: Markus Vieth <39675748+mvieth@users.noreply.github.com>
Co-authored-by: Markus Vieth <39675748+mvieth@users.noreply.github.com>
Copy link
Copy Markdown
Member

@mvieth mvieth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mvieth mvieth merged commit ec3dee7 into PointCloudLibrary:master Sep 19, 2022
@keineahnung2345 keineahnung2345 deleted the frustum_infinite_far_plane branch September 20, 2022 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: new feature Meta-information for changelog generation module: filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants