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

Added example to show Wi-Fi and a subscriber. #131

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

AndyBlightLeeds
Copy link

Added an example that shows how to use micro-ROS with a publisher and a subscriber with Wi-Fi on an ESP32.

Added an example that shows how to use micro-ROS with a publisher and a
subscriber with Wi-Fi on an ESP32.
@pablogs9
Copy link
Member

It would be a good idea to add the build of the examples to the CI procedure. Are you open to contribute this @AndyBlightLeeds ? Do you have experience with Github actions?

I've no time even to build this locally sorry, and it would be great to see that everything is building correctly. Let me know if you can, if not I'm merging this blindly.

@AndyBlightLeeds
Copy link
Author

I haven't done any GitHub actions for a while. but I'll see if I can add one for the new example this afternoon. I'll let you know how I get on.

@AndyBlightLeeds
Copy link
Author

Thanks for starting the workflow. I think I have fixed the bug in the publisher example. Hopefully, it will work this time.

@AndyBlightLeeds
Copy link
Author

I'll take another look at this tomorrow.

The publisher example code was failing due to the first 3 lines of the
platformio.ini file being commented out.  I can now build that publisher
example on my PC, so hopefully this time it will build both examples.
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