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

SAC Segmentation of PointCloud finds 'no' lines (in simulations) #174

Closed
MostafaGomaa opened this issue Oct 20, 2019 · 10 comments
Closed

SAC Segmentation of PointCloud finds 'no' lines (in simulations) #174

MostafaGomaa opened this issue Oct 20, 2019 · 10 comments
Assignees
Labels
bug Issue has been triaged and it's a bug help wanted We would like for someone to work on this pinned Issue will not be closed by stale bot

Comments

@MostafaGomaa
Copy link
Member

MostafaGomaa commented Oct 20, 2019

Environment:
Fedora 30
Ros Melodic
Gazebo 9 Simulation

Segmentation of PointCloud into lines (in line-func.h) does not find any lines!

The segmentation calculation is used by LaserLinesThread to find laser-lines from laser point-cloud, as well as, by NavGraphGenerationThread to find obstacle lines in the map from obstacle point-cloud.

The issue could be detected on behavior level as robots repeatedly failing mps-align as it can not match laser-lines to tag-lines (since there is no laser-line data to match).

I am wondering at this point if this workes on any other configuration or in the real world (since the issue should not be related to simulation as far as I understand)

@morxa
Copy link
Member

morxa commented Oct 20, 2019

We haven't migrated the robots to Fedora 30 yet, so it may occur there as well.

@MostafaGomaa
Copy link
Member Author

It is currently suspected that the issue has something to do with pcl1.9.
Pending on testing with pcl 1.8

@MostafaGomaa MostafaGomaa self-assigned this Oct 22, 2019
@MostafaGomaa MostafaGomaa added bug Issue has been triaged and it's a bug help wanted We would like for someone to work on this pinned Issue will not be closed by stale bot labels Oct 22, 2019
@morxa
Copy link
Member

morxa commented Oct 22, 2019

Yes, I can confirm that the issue does not occur with pcl 1.8.

@MostafaGomaa
Copy link
Member Author

An issue describing the problem has been filed
PointCloudLibrary/pcl#3439 (comment)

@vmatare
Copy link
Contributor

vmatare commented Oct 24, 2019

Great, so we can just patch with 36c2bd6

@morxa
Copy link
Member

morxa commented Oct 24, 2019

Already in progress: https://koji.fedoraproject.org/koji/taskinfo?taskID=38524013

@morxa
Copy link
Member

morxa commented Oct 24, 2019

The patch indeed fixes the issue, I've created PRs for the Fedora packages:
https://src.fedoraproject.org/rpms/pcl/pull-request/2
https://src.fedoraproject.org/rpms/pcl/pull-request/3
https://src.fedoraproject.org/rpms/pcl/pull-request/4

If you can't wait for the updates, you can download my locally built RPMs here:
https://thofmann.fedorapeople.org/pcl-1.9/

@morxa morxa self-assigned this Oct 24, 2019
@MostafaGomaa
Copy link
Member Author

Cool, thanks !

@morxa
Copy link
Member

morxa commented Oct 28, 2019

There is now also a COPR until this has been merged in Fedora:
https://copr.fedorainfracloud.org/coprs/thofmann/pcl/

@MostafaGomaa
Copy link
Member Author

This has been tested resolved!
Thanks, @morxa and @vmatare.
I mark the issue as closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue has been triaged and it's a bug help wanted We would like for someone to work on this pinned Issue will not be closed by stale bot
Projects
None yet
Development

No branches or pull requests

3 participants