-
Notifications
You must be signed in to change notification settings - Fork 191
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
add OrganizedStatisticalOutlierRemoval #2602
add OrganizedStatisticalOutlierRemoval #2602
Conversation
with this PR, we can get better object bounding box.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
path='sample/data/baxter_realsense_l515.bag', | ||
url='https://drive.google.com/uc?id=1RKZLA4J2_lzhCtvMTWxGQh4ykVVEeSAd', | ||
md5='4dab24bd6e6a7fd5b9e86d053352cdfd', | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@knorth55 did you update the baxter_realsense_l515.bag
file recently? It seems the file is updated on 11/17
https://drive.google.com/drive/u/1/search?q=baxter_realsense_l515.bag
fixed in 4ce5c33
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry, I update the rosbag file for this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not for this PR, for this PR #2656
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a PR to fix this issue #2751
StatisticalOutlierRemoval
is useful to remove ghost point cloud, PCL1.8 (melodic) does not supportkeep_organized
option.This PR support
keep_organized
option for Melodic + PCL 1.8.Note:
keep_organized
options is supported in abovePCL1.9
, but not supported inPCL1.8
Melodic uses
PCL1.8
, so I implement this PR.PointCloudLibrary/pcl#1663
TODO:
Input of
Statistical Outlier Removal
Output of
Statistical Outlier Removal
@tohirose