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

Incorrect values for wdyk_2p63a_energy_meter (current/power/voltage) #1717

Closed
rbreiff opened this issue Mar 6, 2024 · 2 comments
Closed

Comments

@rbreiff
Copy link

rbreiff commented Mar 6, 2024

The displayed values of current and power (B+C) are incorrect or missing voltage (B+C)

Screenshot 2024-03-06 at 16-45-32 Tuinkamer – Home Assistant

Expected behavior
Value of Current A, B and C in range 0 -20 A,
Value of Power A in Watt,
Value of Power B and C seem to be factor 1000x,
Value of Voltage B and C seem to be missing

@rbreiff rbreiff added the bug Something isn't working label Mar 6, 2024
@make-all make-all added device improvement Improvement to an existing device config and removed bug Something isn't working labels Mar 6, 2024
@make-all
Copy link
Owner

make-all commented Mar 6, 2024

This is a complaint, not a bug report.

Nothing can be done with this information. I need logs or diagnostics showing the exact data coming from the device.

@make-all make-all added the awaiting feedback Needs more information label Mar 6, 2024
@rbreiff
Copy link
Author

rbreiff commented Mar 7, 2024

tuya_local-2d999e37f9b27265873f8add4c1ab0f1-WP (SPRSUN) Energiemeter-6a9445f90048bb02a17480b8c16009a3(1).json
Thank you for the quick reply. I've added the diagnostic log.
Is this OK?

@make-all make-all removed the awaiting feedback Needs more information label Mar 7, 2024
make-all added a commit that referenced this issue Mar 9, 2024
The phase decoding was originally implemented blindly based on other devices
in #659. Doc confirming this format were supplied in #1078, but still no logs.

Based on the diagnostics in issue #1717, only phase A appears to use the
documented format with 2 bytes voltage, 3 bytes each for current and power,
and 2 bytes for residual current.  Other phases appear to drop the residual
current, meaning everything else needs to shift by two bytes.
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release labels Mar 9, 2024
@make-all make-all removed awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release device improvement Improvement to an existing device config labels Mar 27, 2024
timlaing pushed a commit to timlaing/tuya-local that referenced this issue Aug 8, 2024
The phase decoding was originally implemented blindly based on other devices
in make-all#659. Doc confirming this format were supplied in make-all#1078, but still no logs.

Based on the diagnostics in issue make-all#1717, only phase A appears to use the
documented format with 2 bytes voltage, 3 bytes each for current and power,
and 2 bytes for residual current.  Other phases appear to drop the residual
current, meaning everything else needs to shift by two bytes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants