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

Mapping tutorial with SLAM toolbox #80

Merged
merged 3 commits into from
Oct 4, 2022
Merged

Mapping tutorial with SLAM toolbox #80

merged 3 commits into from
Oct 4, 2022

Conversation

knmcguire
Copy link
Collaborator

Hi!

Here a tutorial + examples to do mapping with the slam toolbox and the crazyflie + flowdeck + multiranger as part of this issue #73

A screenshot of the first part of the tutorial:

Screenshot 2022-10-04 110305

and this is the video embedded: https://www.youtube.com/watch?v=-NfKnlJMAHQ (might be updated with the NAV2 tutorial later)

The only problem that I'm having, is that the setup.py does not properly symlink the launch files when doing colcon build --symlink-install .... I couldn't really find an solution to this so it's a bit annoying to constantly build whenever is changed... Maybe change this to cmake too?

@whoenig
Copy link

whoenig commented Oct 4, 2022

Looks great!

Yes, this symlink issue is a known bug in ROS2. The only current known workaround is to use cmake as a build system.

@knmcguire
Copy link
Collaborator Author

yeah I saw that... was hoping that I was able to fix it with setup.py by now but I guess not. I'll convert it to cmake in a later PR then, since the NAV2 also includes some config files

@knmcguire knmcguire merged commit 039d382 into main Oct 4, 2022
@knmcguire knmcguire deleted the slam_tutorial3 branch October 4, 2022 10:45
boomer319 pushed a commit to boomer319/verrueckterschwarm2 that referenced this pull request Nov 25, 2023
Mapping tutorial with SLAM toolbox
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.

None yet

2 participants