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

ROS 2 #25

Closed
wants to merge 11 commits into from
Closed

ROS 2 #25

wants to merge 11 commits into from

Conversation

sacovo
Copy link

@sacovo sacovo commented Mar 16, 2024

I managed to compile and run the software on ros2, but I'm not happy yet with the results, but I'm not sure what the issue is. I can start it up, with the euroc_mav configuration. And then I start to play one of the rosbags, and it starts tracking and also publishes results. But the results are way off, and the covariances are huge.

Maybe I'm doing something wrong with setting the whole thing up, or there seems to be an issue with something I changed in order to make it build on ros2.

I don't have a ros1 setup to compare, which makes this a bit harder.

@sacovo sacovo marked this pull request as draft March 16, 2024 18:53
@sacovo sacovo mentioned this pull request Mar 16, 2024
@sacovo sacovo marked this pull request as ready for review March 17, 2024 20:00
@jacob-02
Copy link

Could you also provide an edited readme on how to use this ROS2 port for the example files mentioned?

Thank you

@sacovo
Copy link
Author

sacovo commented Jul 18, 2024

Hi, I've been using this branch for other stuff, so I'll close this PR and open a new one with a clean history and I'll include a Readme as well.

I'll close this PR and open a new one, I guess I should be able to do this over the weekend.

@sacovo sacovo closed this Jul 18, 2024
@jacob-02
Copy link

Hey, any updates on this regards?

Thanks in advance

@sacovo
Copy link
Author

sacovo commented Aug 7, 2024

See: #38

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.

2 participants