[go: nahoru, domu]

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 Arduino Extension #1204

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Add Arduino Extension #1204

wants to merge 8 commits into from

Conversation

birdup000
Copy link
Contributor
@birdup000 birdup000 commented Jun 7, 2024

Description

This change introduces a new arduino_control class that allows for controlling an Arduino board via USB or WiFi. The class provides methods for sending raw commands, reading output, performing digital and analog reads/writes, setting pin modes, and controlling an LED.

Motivation and Context

This change is required to enable the ability to control and interact with an Arduino board from within the application. It solves the problem of having to manually interface with the Arduino, allowing for programmatic control and integration with the application's functionality.

How has this been tested?

The changes have been tested by connecting an Arduino Mega board to the development environment via USB and WiFi. The various methods of the arduino_control class were tested by sending commands and verifying the expected behavior on the Arduino board. Additionally, unit tests were added to ensure the proper functioning of the class methods.

Screenshots (if appropriate)

N/A

Types of changes

Changes visible to users:

  • New feature (prefix: feat - non-breaking change which adds functionality)

Internal changes:

  • Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
  • Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

  • My change requires a change to the documentation.
  • My change works!

@birdup000
Copy link
Contributor Author

Wait until I keep testing XD don't wanna send breaking changes in the extensions

@Josh-XT Josh-XT marked this pull request as draft June 7, 2024 10:11
@birdup000 birdup000 marked this pull request as ready for review June 8, 2024 03:31
@birdup000 birdup000 marked this pull request as draft June 13, 2024 05:10
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.

None yet

2 participants