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

Last inverter missing Alarms informations [Bug] #1105

Closed
1 task
DejanBukovec opened this issue Aug 19, 2023 · 24 comments
Closed
1 task

Last inverter missing Alarms informations [Bug] #1105

DejanBukovec opened this issue Aug 19, 2023 · 24 comments
Assignees
Labels
bug Something isn't working stale closed to no response / progress

Comments

@DejanBukovec
Copy link

Platform

ESP32

Assembly

I did the assebly by myself

nRF24L01+ Module

nRF24L01+ plus

Antenna

external antenna

Power Stabilization

Elko (~100uF)

Connection picture

  • I will attach/upload an Image of my wiring

Version

0.7.36

Github Hash

ba218ed

Build & Flash Method

AhoyDTU Webinstaller

Setup

Debug Serial Log output

No response

Error description

Hi,
Is possible that there is bug in new "Alarms" function?
I have 3 inverters and for example today first two of them get information of "Alarm" and count is 1 and last one do not get information and it stay at 0...

slika
slika

slika
slika

Problematic one:
slika
slika

@DejanBukovec DejanBukovec added the bug Something isn't working label Aug 19, 2023
@DejanBukovec DejanBukovec changed the title [Bug] Last inverter missing Alarms informations [Bug] Aug 19, 2023
@Ollipop030
Copy link

Ist mir gar nicht aufgefallen, aber bei mir hat der letzte Inverter auch keinen Alarm. Sollte ja zumindest der Start heute morgen drin sein.

@knickohr
Copy link

knickohr commented Aug 20, 2023

Nein, das ist nur blöder Zufall.

Könnte auch behaupten, der erste Inverter ist der der am letzten online kommt. Das beobachte ich bei meinen 3 DTUs 😉

Es werden die letzten 10 Alarms vorgehalten.

Aber man sollte noch das Datum mit aufnehmen. Verwirrt manchmal wenn mehrere Einträge drin sind, vor alles sieht es dann doppelt aus, falls der gleiche Alarm am nächsten Tag nochmal kommt.

@DejanBukovec
Copy link
Author

DejanBukovec commented Aug 20, 2023

I can confirm that today again Inverter 2 didn't get Alarm data and stay at 0. Inverter 0 and 1 get one alarm(Inverter start) and this is ok. Also it looks like it is old data because time is exactly same as yesterday which is not possible... By experience with OpenDTU every day Alarms reset and I receive new Inverter Start and other Alarms...
Question now is also why there is no new data(Inverter Start)? Is possible that Inverter start producing power before DTU start pooling and DTU do not accept data? Does DTU periodically pool Alarm data(Read also old data from inverter)?

@knickohr Using multiple AhoyDTU's at same time for same inverters as I know is not ok and produce communication problems...

@ghost
Copy link

ghost commented Aug 20, 2023

Good morning,
I also have two inverters, HM-300 & HM-600, ESP 3266, works as expected, both inverters show their start alarms.
( except #1096)
regards Herman

@DejanBukovec
Copy link
Author

DejanBukovec commented Aug 20, 2023

Im remove AC power and now for Inverter 0 and 1 I have 5 Alarms and for inverter 2 I have 4 Alarms...

slika
slika
slika

I think that Sunrise/Sunset function pooling do not work ok and my inverters start before location sunrise and work longer than location sunset... For Sunset Im 100% sure because I see that inverters has produce small power when DTU already stop pooling(I have enabled "Pause polling inverters during night")... Then Im also set "Offset (pre sunrise, post sunset)" to 10 minutes but I will now try 1 hour to see if tomorrow DTU will catch Alarms...

@ghost
Copy link

ghost commented Aug 20, 2023

Hi, correct me if I'am wrong but:
Inverter starts have nothing to do with DTU start or pooling and readout.
as far as you don't switch the mains of the inverters separate! (Shelly?)
The fact that you get multiple notifications is allready in the pipeline (#1096 )
regards Herman

@DejanBukovec
Copy link
Author

@Peertje62 I don't have problems like you... I completly understand my events... It looks like AhoyDTU keep in memory old data...
Inverter 0:
Inverter start 7:02:33 is YESTERDAY start event
Inverter start 7:06:35 is TODAY start event
Other 3 events happen when Im remove AC power.

1 hour ago Im reboot DTU an now Im lost YESTERDAY events and see 4 Alarms for ALL 3 Inverters:
slika
slika
slika

Now I have 60 minute offset and will see if geting alarms(Problematic Inverter Start) from Inverter have some connection with pooling... I think that it have so if pooling is disabled for inverter you will not receive Alarms until some other alarm happen(Which normaly do not happen)...

@ghost
Copy link

ghost commented Aug 20, 2023

Reboot Ahoy at midnight

@knickohr
Copy link

knickohr commented Aug 20, 2023

Yes, Sunrise, Sunset can be an issue if it is too Late/early.

No, I‘m using different Inverters on the different DTUs 😉

@DejanBukovec
Copy link
Author

@Peertje62 Can you please tell me how reset at midnight will solve that DTU do not pool alarm data in the morning? Only thing which will help will be reset after all inverters are online…

@knickohr ok understand you use something like one DTU for one inverter… Does this help you with “power change”(For example zero feed option) response times?

@lumapu can you please check code if is possible to add read alarms when inverters became online? Thanks.

@ghost
Copy link

ghost commented Aug 20, 2023

Hi,
the assumption was that the DTU was showing the old alarm messages again as they were apparently stored in the DTU, a reboot would fix that

@DejanBukovec
Copy link
Author

@Peertje62 Im open issue about missing alarms and not duplicated or old data... For old data is already opened issue #1096 which is different as my problem...

@lumapu
Copy link
Owner

lumapu commented Aug 20, 2023

@DejanBukovec can you post a log which was captured during inverter start? Maybe a Ahoy reboot is enough.

lumapu added a commit that referenced this issue Aug 20, 2023
* reset alarms at midnight (if inverter is not available) #1105, #1096
* add option to reset 'max' values on midnight #1102
* added default pins for CMT2300A (matching OpenDTU)
@lumapu
Copy link
Owner

lumapu commented Aug 21, 2023

do you use power limit control? Checking the code I saw, that once you apply a new power limit control command the entiere queue will be cleared. If this happens before the alarms were read the read will not be triggered again.
Sure it's a problem of design - I will work on it.

@DejanBukovec
Copy link
Author

DejanBukovec commented Aug 21, 2023

Today with 60 minute offset it didn't receive alarm data.
After reboot Alarms stay at 0.
I will try later connect PC to DTU and get logs after reboot... Because there will be a lot of data is there some specific string/hex which I need to wait?

I use power limit because have "zero export" solar system So I send probably imediately when inverter became online power limit command which reset alarm receive command... Can you somehow re-add alarm command together with statistics command into queue if it is deleted from queue?

One question about queue. Is there some optimization of queue request commands in queue?
For example there is queue:
set inverter 1 power 100 - This command is currently sending
set inverter 2 power 105
set inverter 3 power 102
set inverter 1 power 205
set inverter 3 power 102
set inverter 1 power 300
set inverter 2 power 450
set inverter 3 power 200

After optimization and when command "set inverter 1 power 100" will finish it will became:
set inverter 1 power 300
set inverter 2 power 450
set inverter 3 power 200

@knickohr
Copy link

With the v0.7.39 all of my Inverters now have a Start Alarm.

@DanielR92
Copy link
Collaborator

Is that fixed now?

@DejanBukovec
Copy link
Author

No it is not. I m report this issue with latest release 0.7.36 and there is no new release yet...

@DanielR92
Copy link
Collaborator

Thanks, then I try to solve it. :)

@DejanBukovec
Copy link
Author

Im yesterday install prebuilded dev version 0.7.40 and problem is not fixed in it. Alarm for Inverter Start is still missing(Cleared from queue because of limit commands)...

@DejanBukovec
Copy link
Author

Today in the morning when Inverters already produce some small power I'm install prebulded dev version 0.7.41 and issue is not fixed. Only first inverter has get Alarm data after DTU reboot...

lumapu added a commit that referenced this issue Sep 17, 2023
* only request alarms which were not received before #1113
* added flag if alarm was requested but not received and re-request it #1105
@stefan123t
Copy link
Collaborator

@DejanBukovec Danke für die Meldung, ja es gibt noch ein paar Unbekannte / unknown Fehlermeldungen.
U.a.

Ich bin in den assets/warn_code.json der Hoymiles S-Miles Installer App fündig geworden. 🥳

Hier mal die bisher aufgetretenen Fehler oder Warnings aus dem in #121 angehängten warn_code.json (GitHub versteht nur ZIP ?).

11: Grid voltage surge
12: Grid voltage sharp drop
13: Grid frequency mutation
14: Grid phase mutation
36: INV overvoltage or overcurrent
46: FB overvoltage
47: FB overcurrent
71: VW function enable
72: FW function enable
73: TW function enable

Es wird von @lumapu im Rahmen von #1105, #177 oder #608 in Ahoy implementiert und daher kann dieses Issue evtl. auch bald geschlossen werden ?

@MrFr33man123
Copy link

Ich versuche mich mal an das Issue ran zuhängen. In einer älteren AHOY Version habe ich hochzählende Nummern als Alarm bekommen. Jeden Inverter Start von der Nummer des letzten wieder hoch.
20 => 21 => 24 => 29 => 30 => 31 => ...
Aus den Daten hatte ich abgeleitet das die Auswertung den Unterschied zwischen der Vorherigen Nummer und der Aktuellen die Eigentliche Nachricht ist. Für meinen HM-600 war das:
0 | Inverter Start
1 | Grid OK Starting Production
3 | Producing Power
5 | Grid Connected

Mit der aktuellen Version bekomme ich immer nur die Erhöhung um 1.

Was wird denn aktuell genutzt als Ausgabe?

@lumapu
Copy link
Owner

lumapu commented Oct 15, 2023

die ID ansich sagt nichts zum Alarm, sie gibt nur an wie viele Alarme getrackt wurden. Anhand dieser ID liest Ahoy dann die Alarme der Reihe nach aus.

@lumapu lumapu added the stale closed to no response / progress label May 30, 2024
@lumapu lumapu closed this as completed May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working stale closed to no response / progress
Projects
None yet
Development

No branches or pull requests

7 participants