-
Notifications
You must be signed in to change notification settings - Fork 92
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
firmware error #35
Comments
use the |
so it doesn't work on foxy |
The "galactic" branch of this repository works on ROS humble. I think it will on ROS foxy as well. The branch named "foxy" is broken. |
how do i use the foxy branch, ros2 foxy is the only foxy i have right now |
I don't believe the "foxy" branch of this repo works right now, but the "galactic" branch of this repo will likely work with the ROS Foxy on your system, regardless of the different names. |
so i need to install ros 2 galactic to get galactic branch |
no. Just checkout the "galactic" branch of this repository.
|
I see. I have a guess on how to fix it, but can't guarantee it will work because I haven't tested myself. In Looking at last line of the the traceback, it mentions line 96 of The change I'm asking you to make to This would also explain why it worked for me with humble and rolling, as those aren't deprecated by micro_ros_platformio. Let me know how that goes, and I'll try to help troubleshoot further if needed. |
I can reproduce your errors (galactic branch, ROS foxy WSL Ubuntu 20.04):
After I make the change I suggested in firmware/platformio.ini, I get the following
I'll try to troubleshoot further |
I got it to compile after some more troubleshooting. This repo needs some work to support different ROS distros and test them - I'll try to get that working if I can get PRs merged. In the meantime, @justinbar here's the workaround: I found this existing issue #13 my
I didn't change anything above or below - just changed lib_deps
this worked for me:
|
think you it worked! so far I was able to drive my robot around. This project need to be up to date due these type of errors. |
Glad to have helped! And agreed - there are a number of issues to be resolved. I only found this repo recently myself, but I made PR #34 to begin automatic testing for stuff like this, and if it gets merged by this repo's owner I plan to move on to fixes for issues like this. |
@justinbar can you try checking out the updated foxy branch, pulling in changes, and confirm if it works now? Shouldn't need to tinker with anything. If it does work, do you mind closing this issue? Thanks ! |
it worked for foxy branch |
@justinbar Wonderful. Can you close the issue? |
when i compile the firmware teeny40 i keep on getting this error
The text was updated successfully, but these errors were encountered: