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 current_resistor and voltage_divider configuration to CSE7766 platform #736

Open
DanielWe2 opened this issue May 18, 2020 · 1 comment

Comments

@DanielWe2
Copy link

DanielWe2 commented May 18, 2020

Describe the problem you have/What new integration you would like

The CSE7766 platform should have options to configure the values for current_resistor and voltage_divider. (Similar to the one for HLW8012)

Please describe your use case for this integration and alternatives you've tried:

The current implementation assumes that all devices using the CSE7766 chip use the default values for the current sense resistor and the voltage divider resistor as proposed in the data sheet.

Devices like the Blitzwolf SHP5 or the Gosund SP112 (at least in some markets) use different resistors. You will get very wrong values for voltage and power with the default.

Mine uses two 953kΩ resistors in series (1906 kΩ) for voltage sensing. The current code assumes 1 MΩ.

You could work around it with calibrations but that is a time consuming process. It would be way better to fix the root cause and we would have simple pair of values to share with others with the same device to fix their values. Also: You would need calibrations for power, voltage and current.

@TeHashX
Copy link

TeHashX commented Apr 14, 2022

This will save a lot of time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants