DTU polling interval is no longer polled after setting set.

[DanielR92]

What happened?

If I set the DTU interval from 5 seconds down to 3 seconds during operation... then I no longer receive any data.
If I restart the DTU, I will receive data again at shorter intervals than set.

To Reproduce Bug

After a restart, you setup the settings to a new value. The issue will come bag.

Expected Behavior

After setting the value, the new legal time should be recognized as the maximum.

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

gad493c0

Relevant log/trace output

No response

Anything else?

No response

Please confirm the following

• I believe this issue is a bug that affects all users of OpenDTU, not something specific to my installation.
• I have already searched for relevant existing issues and discussions before opening this report.
• I have updated the title field above with a concise description.
• I have double checked that my inverter does not contain a W in the model name (like HMS-xxxW) as they are not supported

[schlimmchen]

I cannot recall that this ever happened to me, and from looking at the code for 20 min I am confused as to how this could happen at all. I had hoped for an easy explanation.

@spcqike @Manos1966 can you guys reproduce this?

[Manos1966]

Ich werde es morgen testen
(ich habe das DTU interval seit 2023 nicht angefasst 😜 )

[spcqike]

I can’t. Changing from 1 to 3 to 2 seconds, everything works as expected without reboot.

[Manos1966]

git-hash/version of OpenDTU = gad493c0

Welche Version ist diese? Eine ältere?

[DanielR92]

Das müsste dieser hier sein: https://github.com/helgeerbe/OpenDTU-OnBattery/actions/runs/10652874817?pr=1216

[Manos1966]

Aha! Dann macht es also Sinn, warum unsere offizielle Versionen dieses Problem nicht haben.

[DanielR92]

Ja @schlimmchen hat nach Testern aufgerufen und nun bin ich hier. :)
PS: Wenn ich in openDTU in der Systemseite den git-Hash anklicke, leitet es mich hierhin weiter...
https://github.com/helgeerbe/OpenDTU-OnBattery/releases/tag/gad493c0

Aber diese Seite existiert nicht mehr.

[DanielR92]

I can’t. Changing from 1 to 3 to 2 seconds, everything works as expected without reboot.

You don't understand me correctly, in my version what I have. It is necessary to reboot manually to work with the new settings.

[schlimmchen]

Aber diese Seite existiert nicht mehr.

@DanielR92 Das hat damit zu tun, dass ich den commit weggebügelt habe mit einem force push. Damit musst du bitte leben, dass das passieren kann, wenn du Vorabversionen aus Pull Requests installierst.

@spcqike @Manos1966 Thanks for looking into it! Lets keep an eye on it, but it seems this is a fluke. Daniel has multiple of those, but they are real, he showed me by sharing his screen using discord, so there should be something to it...

[DanielR92]

@schlimmchen Hab doch nichts dagegen gesagt... ;)

[schlimmchen]

Yesterday I compared the sources of lib/Hoymiles/ and src/WebApi_dtu.cpp between this project and the upstream, but could not find anything unexpected or wrong.