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

Add OpenCyphal Node #3

Merged
merged 9 commits into from
Sep 12, 2024
Merged

Conversation

manforowicz
Copy link
Contributor

This pull request:

  • Enables the ESP32 to act as a simple OpenCyphal node if enabled in the web interface. This functionality is disabled by default. The OpenCyphal node broadcasts a heartbeat every second, and counts the number of heartbeats received.
  • Adds collapsible descriptions to some of the setting fields in the web interface. This should make it easier to understand what those fields mean.
  • Moves the CAN listening logic from socketcand_server.c to the new can_listener.c. This makes it possible for other tasks, such as in cyphal_node.c, to also listen to the CAN bus.
  • Gets rid of the socketcand_translate_frame type, and instead uses the twai_message_t type everywhere.
  • Adds some additional warning flags to GCC.
  • Splits the .github workflow into build.yml and release.yml. build.yml uses the latest version of ESP-IDF, whereas release.yml uses a specified stable version of ESP-IDF.
  • A few minor changes that simplify code, or improve behavior.

@wiboticalex wiboticalex merged commit 4cac9d2 into wibotic:main Sep 12, 2024
1 check passed
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