Legacy Smarthome shelly fix

[gvzdus]

Discussion in https://forum.openwb.de/viewtopic.php?p=134286#p134286

• Supports Shelly EM Gen3 (S3EM-002CXCEU)
• Supports HTTP-Digest (needed for password protected >= Gen2 devices)
• Better logging
• Caching of components and /status & /rpc/Shelly.GetStatus response for 5 seconds for multiple counters on the same device
• Correcting wrong channel interpretation (real channel 0 needed param value 2 instead of documented param value 1)

Tested with 12 shelly devices

[LKuemmel]

@gvzdus

[quote=Thrillhouse post_id=136940 time=1764671323 user_id=4334]
ich bin vom derzeitigen Release zum aktuellen Master gewechselt und musste feststellen, dass die OpenWb keine Daten mehr vom shelly em 50 angezeigt hat. Nachdem ich wieder zurück zum Release gewechselt bin, wurden die Daten wieder angezeigt.
Der Shelly EM 3 wurde meines wissens von OpenWB "aufgenommen". Ist das beim shelly em 50 demnach nicht so?
[/quote]

LenaK hat meinen PR für breitere Shelly-Smarthome(!)-Unterstützung gemerged.
Eine Änderung ist wichtig:
[b]Die Kanal-Nummern sind jetzt so wie dokumentiert![/b]
Bedeutet: Aufgrund eines Bugs musste man für verschiedene Shellys "immer eine Kanal-Nummer höher" angeben: "Kanal 0 / Meter 1" gab es de fakto gar nicht, "Kanal 1 / Meter 2" entsprach dem ersten Kanal. Jetzt sollte es richtig sein, aber man muss eben die Kanalnummern anpassen.

Wegen meines PR fühle ich mich jetzt aktuell für den Smarthome-Shelly-Support verantwortlich. Sollte es neben der beschriebenen Kanalnummern-Änderung andere Probleme geben, kümmere ich mich gerne darum. Hilfreich wäre die Info, ob Du Zugriff auf die Infos auf der Ramdisk, wo Device-Infos gecachet werden, hast. (Shell-Zugang).

Die Korrektur der Kanal-Nummern sollte automatisch über den upgrade-datastore erfolgen, sonst bekommen wir viele solcher Meldungen. Kannst Du bitte mal das Config-Topic mit den Kanal-Nummern posten, wie es vorher aussah und wie es jetzt aussehen muss?

[gvzdus]

Es geht um:
openWB/LegacySmartHome/config/get/Devices//device_chan

0 => 0 (any)
1 => 1 (dürfte nie funktioniert haben)
2 => 1
3 => 2

Schwierig ist natürlich: Falls "upgrade-datastore" (kannte ich nicht) mehrfach ausgeführt würde, wird auch jedes Mal die Kanalnummer niedriger. Und, um es noch schwieriger zu machen: M.W. gab es den "immer eins mehr als in der Config-Seite suggeriert"-Bug nicht bei allen Shelly-Generationen - da muss ich noch mal in den Alt-Code gucken.

[gvzdus]

Ich habe noch mal in den Code geguckt:
Den (alten) Bug findest Du in:
https://github.com/openWB/core/blob/405fadb377ece41183f016d680cbbc5cc5f72a22/packages/modules/smarthome/shelly/watt.py
in den Zeilen 114-117. Diese hätten nie eingefügt werden sollen, und führen dazu, dass es den Wert "1" gar nicht sinnvoll gibt, und "2" auf Kanal "a" u.sw.. Es betrifft m.E. genau alle 3-Kanal-Shelly-Geräte > Gen1 - Gen 1 oder 1/2/4-fach-Meter sollten nicht betroffen sein.
Daher sollte man es m.E. auch nicht automatisch umstellen, denn Du weisst beim Prozessieren der Config ja nicht, welcher Typ von Shelly nun genau dahinter liegt.

[LKuemmel]

Ich habe hier grob beschreiben, wie upgrade-datastore funktioniert: https://github.com/openWB/core/wiki/Neues-Modul-programmieren#breaking-changes-und-ergänzen-von-neuen-einstellungen

Wenn openWB/LegacySmartHome/config/get/Devices/4/device_type == shelly bekommt man über reg.get_http_session().get(f'http://{ipadr}/shelly').json()['gen'] die Generation und aus openWB/LegacySmartHome/config/get/Devices/4/device_chan den Channel, kann ihn konvertieren und wieder an dieses Topic publishen. So müsste es der Code in dem Link auch verarbeiten.

[benderl]

Ich nehme den PR wieder aus dem Master raus, damit es nicht zu kompliziert beim kommenden Datastore Upgrade wird. Wenn jetzt einige Anwender Ihre Konfiguration manuell anpassen, wird die potentiell durch das kommende Upgrade wieder zerschossen.

[gvzdus]

Okay, dann kann ich auch besser testen.
Es gibt noch den Error-Report zum Shelly EM50 von Thrillhouse bzgl. meines PRs - den hatte ich um Logfile / RamDisk-Auszüge gebeten, aber noch kein Feedback erhalten. Daher ist aber nicht ausgeschlossen, dass ich noch was anderes fixen muss.