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

Usermod INA219 Current, Voltage & Power Sensor #4237

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

Conversation

KrX3D
Copy link

@KrX3D KrX3D commented Oct 31, 2024

Hi,

i created a usermod for the ina219 Current, Voltage & Power Sensor:

INA219-Pin-Outs

I took a liking to the INA226 Usermod here:

https://github.com/Aircoookie/WLED/tree/0_15/usermods/INA226_v2

and took some parts and added my 3 cents to it.

I hope the readme states everything needed to get what can be done.

the only think i couldnt solved is to add the sensors data to the wled home assistant integration page (if thats even possible) so instead it is added just to mqtt. if there is a way to have the autodiscover to add it directly to the wled integrations page, i would be more than happy to change the usermod.

also i hope i added everything correctly in git, since its my first time working like this in git

@DedeHai
Copy link
Collaborator

DedeHai commented Nov 1, 2024

since you keep adding commits, is this work in progress? if so, mark it as such or better yet: commit to this PR once you are finished.

@KrX3D
Copy link
Author

KrX3D commented Nov 1, 2024

since you keep adding commits, is this work in progress? if so, mark it as such or better yet: commit to this PR once you are finished.

Hi,

no its finsied. I just did some small formatting changes while waiting for a reply. But now im happy with it ;)

EDIT: im also using it for like 2 weeks now without any problems

@DedeHai
Copy link
Collaborator

DedeHai commented Nov 1, 2024

ok nice. did you test it working on different ESPs?

@KrX3D
Copy link
Author

KrX3D commented Nov 1, 2024

i tested it on an ESP32 S2 (lolin_s2_mini) and an C3 (esp32c3dev) where it is working.

like i said its already running for around 2 weeks

@KrX3D
Copy link
Author

KrX3D commented Nov 14, 2024

Hi,

i tested it now also with an ESp32 Wroom-32 (esp32dev) and it also worked.

did i forget something so it will be added to version 0,15?
im new to git with all this pull request stuff

@DedeHai
Copy link
Collaborator

DedeHai commented Nov 14, 2024

looks fine, needs more testing and review, there are a few usermods in pipeline so please be patient.

changed uint8_t to int8_t for _sdaPin and _sclPin --->  to be able to pick "unused" in pin selection which is -1,
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