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

EDL21 - Wrong "Sum active instantaneous power" values #92776

Closed
olu12345 opened this issue May 8, 2023 · 3 comments · Fixed by #93035
Closed

EDL21 - Wrong "Sum active instantaneous power" values #92776

olu12345 opened this issue May 8, 2023 · 3 comments · Fixed by #93035

Comments

@olu12345
Copy link

olu12345 commented May 8, 2023

The problem

Hello everyone,

I have the problem that as soon as the "Sum active instantaneous power" values ​​become negative, they jump to over 160,000W (positive value).

I have a photovoltaic system and during the day the values ​​are negative.

What version of Home Assistant Core has the issue?

core-2023.5.2

What was the last working version of Home Assistant Core?

core-2023.4.5 or below

What type of installation are you running?

Home Assistant OS

Integration causing the issue

edl21

Link to integration documentation on our website

https://www.home-assistant.io/integrations/edl21/

Diagnostics information

home-assistant_edl21_2023-05-08T08-34-31.641Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

See line number 220, the value is at 164488.35 W.  

The correct value should be approximately -2500W

Additional information

The Meter is a DZG DWS7420.2.G2

@olu12345 olu12345 changed the title EDL21 - Negative "Sum active instantaneous power" error EDL21 - Wrong "Sum active instantaneous power" values May 8, 2023
@mtdcr
Copy link
Contributor

mtdcr commented May 13, 2023

Can you try to downgrade pysml to version 0.0.9 to see if it fixes the problem for you? I suspect that the workaround that was included in 0.0.10 for some DZG meters causes it.

@mtdcr
Copy link
Contributor

mtdcr commented May 13, 2023

I was able to reproduce the problem based on your log and released pysml version 0.0.11. It restricts the newly implemented DZG workaround to meters with serial numbers below 60000000: mtdcr/pysml@7c96d22?diff=unified

This is probably going to fix the problem for you, but in case 60000000 isn't the correct limit between devices containing the bug (probably pre-G2) and those without (probably G2), this value might need to be adjusted in the future to cover other users. It may also turn out to be unfeasible to use a single serial number to detect the presence of this bug.

@mtdcr
Copy link
Contributor

mtdcr commented May 13, 2023

@StephanU tagging you, to make you aware of this issue and the new version of pysml, because you handled the previous updates.

@StephanU StephanU mentioned this issue May 13, 2023
20 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Jun 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants