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

read alert field #126

Closed
CFenner opened this issue Sep 5, 2022 · 7 comments · Fixed by #149
Closed

read alert field #126

CFenner opened this issue Sep 5, 2022 · 7 comments · Fixed by #149
Labels
new-feature New features or options.

Comments

@CFenner
Copy link
Contributor

CFenner commented Sep 5, 2022

I would like to see the alert field in home assistant. Any doubts about adding this?

Detailed description

I would like to read webdata_alarm and make it available as a field in HA.

var webdata_total_e = "0.3";
var webdata_alarm = "F13";
var webdata_utime = "1";
var webdata_alarm = "F06F11F13F23F24F26F27F31";

I'm using Bosswerk MI300 / Deye SUN300G3-EU-230. Is this also available on other inverters?

Context

This would be nice to have for diagnostics.

@robbinjanssen
Copy link
Owner

I see the same field in my JS source, so we can probably add it. We need to add it in the python-omnikinverter library first though :)

@robbinjanssen robbinjanssen added the new-feature New features or options. label Sep 6, 2022
@klaasnicolaas
Copy link
Collaborator

We are working on it, it's just a puzzle to find out per source_type where this value is.

@MarijnS95
Copy link
Collaborator

MarijnS95 commented Sep 6, 2022

@CFenner Do you have a mapping of these error codes? According to certain Omnik docs like https://download.solarclarity.nl/2015/04/UserManual-OMNIK-3K4K5K-TL2.pdf F13 means that the device is overheating...

If we can find a consistent mapping for all these devices we might be able to provide a friendly representation in the UI, instead of raw text.

I'll check if I can find an error-code mapping in the TCP stream in the meantime.

@CFenner
Copy link
Contributor Author

CFenner commented Sep 6, 2022

No, I don't have a mapping. It's definitely not overheating, current production is very low. I asked Deye for it.

@robbinjanssen
Copy link
Owner

robbinjanssen commented Sep 6, 2022

So we found a few sources that have some mapping, for example: https://download.solarclarity.nl/2015/04/UserManual-OMNIK-3K4K5K-TL2.pdf

Screenshot 2022-09-06 at 15 09 52

or in dutch: https://www.solar-nu-webshop.nl/data/upload/files/1911-omnik-handleiding-01-usermanual-omnik-single-phase-inverter-installdutch.pdf

screenshot_2022-09-06_at_15 12 07

But if it doesn't match your current problem, the mapping is useless.

@MarijnS95
Copy link
Collaborator

@CFenner How useful is this field without knowing what's up?

I can see use in knowing the value, but wouldn't want to unnecessarily scare users because of some non-empty error code/value.

@github-actions
Copy link

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

@github-actions github-actions bot added the stale label Sep 21, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Mar 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-feature New features or options.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants