-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
[filters] CropHull unexpected number of intersections #3912
Comments
There's no need to tag the maintainers on the issues. |
As per the code, If a ray intersects with an even number of polygons, it is outside the hull, else it is inside. I don't see where this number would be limited to 0, 1 and 2 for a concave hull. If the hull was convex, then it's a bug to have such a high number of crossings. Could you replicate this issue using 1 point and a simple hull? |
The hull I made is from the point cloud of a sphere so wether it is convexe or concave has no importance here. Turned out however that I had a few self intersecting faces, which explained why I could have more than 2 intersections per point. I did the same test with a hull having no self-intersection (checking with blender that geometry is manifold). I uploaded a file convex_hull.ply which contains this hull. I still get the following result for the same point :
|
Marking this as stale due to 30 days of inactivity. It will be closed in 7 days if no further activity occurs. |
I tried to debug the issue and came with with no sane reason why this is happening |
Context
I am using CropHull to test if a single point is inside the hull of a sphere. In
applyFilter3D(PointCloud &output)
function ofcrop_hull.hpp
, I get unexpectedly high number of intersections for the given point.Here is what I added to the function (using this fix) :
Current Behavior
This is the resulting number of intersections I got for each ray :
Expected behavior
I expected crossings array to contain either 0 (no intersection, possibility that the point is outside), 1 (1 intersection, possibility that the point is inside) or 2 values (2 intersections, possibility that the point is outside of the sphere).
To Reproduce
I use PCL 1.8.0 on Windows 10. Here is the link to the files used :
@taketwo @kunaltyagi
The text was updated successfully, but these errors were encountered: