-
Notifications
You must be signed in to change notification settings - Fork 4
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
Can be used on Raspberry Pi Pico (RP2040) ? #2
Comments
To be compatible it needs to have the corresponding modules, including Sadly I don't have a raspberry pico to test it. If you make it compatible, feel comfortable to make a pull request. |
I've been working to migrate my robot to use an RP2040 based board, the Pimoroni Yukon. There are two lines that need to be changed for it to work at a basic level, I've not got the whole thing working with other message types than the ColorMsg example yet though. Lines 17 and 63 in core.py: need to be changed to: The RP2040 uses the same |
So having given up to write my own thing I hit a bunch of problems I realised this has already solved, rather than figuring out how to solve them in my code I've revisited this and got it working. It seems I actually had it working before but just misunderstood how the MD5 hash generation worked. It sounds obvious but the Msg files need to match exactly, as the msg files included as examples didn't have comments I manually removed the comments from mine as I mistakenly believed comments were ignored. A bad assumption on my part! I haven't got more complex messages working yet, like anything with a header, but slowly working through it all. |
Another update in case anyone ends up here and hits a similar problem. It turns out the ROS message generator does ignore comments for md5 generation but the message generator here doesn't, not yet anyhow... Just make sure you manually remove comments before uploading to your MCU and you should be fine. |
Thank you so much for your lib: I was wondering if it can be used on Raspberry Pi Pico (RP2040) with MicroPython.
Has anyone already tried ?
Best regards
The text was updated successfully, but these errors were encountered: