search

fhem / SD_Keeloq

FHEM - Module
https://fhem.de/
3 stars 2 forks source link

Änderung Update-Mechanismus bei Einzeilig #15

Open bismosa opened 5 years ago

bismosa commented 5 years ago

Siehe https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro/issues/11

HomeAutoUser commented 5 years ago

Bitte aufpassen, auch hier https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro/pull/16 das der Grundcode https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro sich geändert hat nach kleinen Fixes und Behebungen. Wir kommen sonst auf Konflikte zu ;-)

HomeAutoUser commented 5 years ago

@bismosa wenn du wieder die Zeit findest, so würde ich dich mal bitten dies https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro/tree/fix_UI zu testen. Ich habe mal herumgespielt und vergleichen. Mit dem sehr geringeren TimeOut, so wird das wahrnehmbare flackern fast unsichtbar hier.

Ich habe das System auch "nur" auf einem Pi und wir sollten ja immer auch nutzen zum Zweck sehen. Ja, wir suchen noch den Weg um es anders zu updaten aber wieso sollten wir den Code noch mehr größer sprengen wenn man es kurz haben kann. Dagegen sollten wir auch schauen, wieviel User nutzen das UI Einzeilig? Stets aber auch im Kopf haben, das das Modul selbst wachsen kann wenn neue Geräte hinzukommt, da es ja schon gerade nicht nur für Jaro geschrieben ist.

Alles Punkte welche wir abwägen sollten.

PS: Eine Diskussion ist keine Kritik :-)

bismosa commented 5 years ago

Hallo! Eigentlich könnte es mir ja egal sein...ich werde die einzeilige Variante vermutlich nie selbst nutzen. Ich denke aber das auch diese Variante gerne benutzt werden wird...deswegen ist mir das dann doch nicht so egal :) Wenn es als einziges Device im Raum ist, ist der Reload bei mir ebenfalls schnell. Wenn ich das im gleichen Raum habe wie auch meine anderen 4 Fernbedienungen (mit mehrzeilig) ist der Reload schon deutlich spürbar. Wenn ich dann noch mein SVG im Raum habe ist der Reload mit über 3sek. nervig.

Ich hatte da auch noch eine Idee, die ich jetzt mal ausprobiert habe. Mal als Diskussionsgrundlage. https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro/pull/18

Hier wird jetzt bei der Änderung der Auswahl nur noch der Wert im Reading gemerkt, aber keine Weitere Aktion ausgeführt. Wenn man einen Button betätigt, wird per JavaScript der FHEM Befehl abgesetzt...dabei wird vorher der Kanal aus dem Optionsfeld hinzugefügt. Damit entfällt der Reload. Bei einem SeitenReload wird der zuletzt eingestellte Kanal wieder übernommen...und in anderen Fenstern bleibt es bei der Einstellung, die auch zumindest in meinem Test problemlos funktionierte.

Lässt sich bestimmt noch verbessern. Kann ich Anfang der Woche mal probieren. Ist von Unterwegs nicht so einfach :)

Gruß Bismosa