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

Arduino Utils #179

Merged
merged 17 commits into from
Jul 4, 2022
Merged

Arduino Utils #179

merged 17 commits into from
Jul 4, 2022

Conversation

leloup314
Copy link
Member

This PR adds (and reorganizes) Arduino-based utils with firmware, software and examples.
The SiLab relay board HL was renamed from Arduino.py to arduino_relay_board.py.
The following Arduino-based utils are added:

  • SerialToI2C (basil/HL/arduino_serial_to_i2c.py): A HL allowing to interface an I2C bus using the Arduino over serial
  • NTCReadout (basil/HL/arduino_ntc_readout.py): A HL allowing to read out up to 8 NTCs in a voltage divider configuration

The corresponding firmware is found in basil/firmware/arduino

@codecov
Copy link

codecov bot commented Jun 30, 2022

Codecov Report

Merging #179 (95c9742) into master (37d6ed8) will decrease coverage by 0.71%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
- Coverage   38.82%   38.10%   -0.72%     
==========================================
  Files          74       77       +3     
  Lines        4886     4978      +92     
==========================================
  Hits         1897     1897              
- Misses       2989     3081      +92     
Impacted Files Coverage Δ
basil/HL/arduino_base.py 0.00% <0.00%> (ø)
basil/HL/arduino_ntc_readout.py 0.00% <0.00%> (ø)
basil/HL/arduino_relay_board.py 0.00% <0.00%> (ø)
basil/HL/arduino_serial_to_i2c.py 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37d6ed8...95c9742. Read the comment docs.

@leloup314 leloup314 marked this pull request as draft June 30, 2022 14:50
@leloup314 leloup314 self-assigned this Jun 30, 2022
@leloup314 leloup314 requested a review from themperek June 30, 2022 14:50
@themperek
Copy link
Member

For me is fine. If you remove the draft it can go I guess.

@leloup314
Copy link
Member Author

Is still need to write some docs and push some changes, when I am done, I mark as ready

@leloup314 leloup314 marked this pull request as ready for review July 1, 2022 11:18
@leloup314
Copy link
Member Author

Go time

@leloup314 leloup314 merged commit 12e998c into master Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants