-
Notifications
You must be signed in to change notification settings - Fork 114
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
Create ROS2 branch #56
Comments
Hi @klintan I've used the ROS-bridge to get the IMU data onto the ROS2 side, it works for me but I don't know exactly what advantages will I get when I use ROS2 driver for the imu sensor. Please share with me your thoughts. Regards, |
In general, the last version of ROS1 is around the corner, then all efforts will be put on ROS2. That is one big reason to port everything needed to ROS2. Further I guess you'll get some overhead in terms of serializing and deserializing if you are using the bridge (probably not a huge problem, depending on your application). Further just having both ROS1 and ROS2 installation, is somewhat of a hassle, launch systems etc to coordinate stuff if used for one robot is not ideal either. The ROS2 version is done though, so feel free to test if you want to :) |
Yes, I got what you mean, you're right. Thanks for the explanation! Thanks for sharing with me the ROS2 version, I'll test it soon and I'll let you know how it will work for me :D |
@klintan it seems not working (ros2-rolling): first of all you have got wrong spelling: secondly, it`s not working with launch:
nor with direct start:
in minicom on port /dev/ACM0 i see dataflow:
ps. fixed only a node (not launch file):
and it seems it`s working by direct start - python3 imu_node.py
|
can you please upload the imu_node.py that is working |
how you solved this issue ? TypeError: argument of type 'NoneType' is not iterable |
I forked this repo solved the problems mentioned and made a pull request for my updates, meanwhile you can check the updates here: |
I'm working on a ROS2 port, would be awesome if you could create a ROS2 branch for future PRs.
Thanks in advance!
The text was updated successfully, but these errors were encountered: