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 Usermod for Power Measurement using esp's adc #4108

Draft
wants to merge 3 commits into
base: 0_15
Choose a base branch
from

Conversation

Tomas-Kuchta-FPV
Copy link

Hi WLED community,

As an intermediate coder and someone who has benefited from this project, I wanted to contribute by creating a usermod that measures power consumption using a voltage divider and a Hall current sensor.

Summary of Changes

  • Implemented functions to measure power consumption.
  • Introduced support for voltage and current measurement using ESP32 ADC.

There are still some issues that I would need some help with

  • I'm having trouble getting the pin manager to work correctly.
  • There may be other issues I'm not aware of.

Any feedback, suggestions, or assistance would be greatly appreciated!

Thank you for your support!

- Implement functions to measure power consumption
@blazoncek
Copy link
Collaborator

Please rebase your PR for 0_15 branch.

- Implement functions to measure power consumption
@Tomas-Kuchta-FPV
Copy link
Author

Hey there,

I'm not very familiar with Git, and it seems like I've messed something up. I was following your guide on how to properly submit a PR in the wiki, but I think I missed forking the other branches. I'd like to start over from scratch.
Thanks for your patience. I'm sorry for any inconvinience.

@blazoncek blazoncek changed the base branch from main to 0_15 August 19, 2024 09:19
@Tomas-Kuchta-FPV
Copy link
Author

Thank you so much for your help should I reverse the 712b72d commit?

@blazoncek
Copy link
Collaborator

Thank you so much for your help should I reverse the 712b72d commit?

No.

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