Entwicklung des Z-Wave JS UI-Dienstes

Hallo,

Versprochen ist versprochen.
Neuer PR verfügbar auf Github: [Zwavejs] Support Air Temperature & Power properties from Multilevel Sensor Command Class by sescandell · Pull Request #2027 · GladysAssistant/Gladys (github.com)

Unterstützung der Temperatur und Rückmeldung der Leistung:

Ich habe eine Frage zum Code und den Parametern @pierre-gilles, bitte: Wofür dient die Eigenschaft has_feedback bei einem Gerät. Beispiel hier:

keep_history und read_only okay, das verstehe ich. Aber der Teil Feedback…

Danke,

@Sescandell
chat-gpt :wink: : also hier das Gerät (das ein Schalter sein muss) sendet, wenn es einen Befehl „von 0 auf 1“ erhält, das Gerät auf 1 und sendet eine Rückmeldung „Befehl ausgeführt, auf 1 gesetzt“ oder eine ähnliche Nachricht

Im Kontext von Z-Wave-Geräten bezieht sich has_feedback wahrscheinlich auf eine Funktion, mit der festgestellt werden kann, ob ein Gerät nach dem Senden eines Befehls eine Rückmeldung erhalten hat.

Das Z-Wave-Protokoll ist so konzipiert, dass es bidirektional ist, was bedeutet, dass Geräte nicht nur Befehle empfangen, sondern auch Rückmeldungen senden können, um anzuzeigen, ob ein Befehl erfolgreich ausgeführt wurde oder eine Aktion durchgeführt wurde. Diese Funktion wird oft verwendet, um die Zuverlässigkeit von Befehlen zu gewährleisten und Informationen über den Status der Geräte zu erhalten.

Wenn has_feedback im Kontext eines Z-Wave-Geräts verwendet wird, gibt es in der Regel an, ob dieses Gerät in der Lage ist, nach der Ausführung eines Befehls eine Rückmeldung zu geben. Dies kann für Smart-Home- oder Überwachungssysteme nützlich sein, bei denen es wichtig ist zu wissen, ob die an die Geräte gesendeten Befehle tatsächlich ausgeführt wurden.

Hallo,

Vielleicht fehlte meiner Frage der Kontext: Es geht um eine Eigenschaft auf Gladys-Seite und nicht um eine spezifische zWave-Eigenschaft.

Ich habe den Code schnell durchgesehen. Ich sehe eine Verwendung bei einem saveState, aber ich bin nicht weiter zurückgegangen. Pierre Gilles sollte in der Lage sein, darauf zu antworten. Falls nicht, werde ich der Sache weiter nachgehen.

Danke für den Versuch :+1:

@Sescandell le has_feedback bedeutet, dass das Gerät am Ende eine Statusrückmeldung sendet, nachdem Gladys ihm einen Befehl gesendet hat.

Dadurch kann Gladys feststellen, ob sie den Wert lokal speichern muss, falls eine Steuerung erfolgt, oder ob sie warten muss, bis das Gerät ein Event sendet, das besagt: „Ich wurde erfolgreich gesteuert“.

Beispiel für has_feedback = true :

Ich steuere eine Steckdose über Gladys → Gladys sendet einen Befehl an die Steckdose → die Steckdose sendet ein Event „state changed“ an Gladys → Gladys speichert den neuen Zustand der Steckdose.

Ich weiß nicht, ob das klar ist :grinning_face_with_smiling_eyes:

Super! Gut gemacht :clap: Ich kann es kaum erwarten, das zu sehen :slight_smile:

Ich weiß nicht, ob du es gesehen hast, aber deiner PR fehlt etwas Testabdeckung?

Alles klar, danke,

Ja, ich habe die Abdeckung gesehen… aber das sind Fälle, die der Linter mich zwingt hinzuzufügen, ohne dass es wirklich etwas bringt… ich werde sehen, was ich damit mache. Ich denke, in diesem Szenario werde ich einen undefinierten Wert zurückgeben, und wenn undefiniert, dann bedeutet das „unbekannte Anfrage“ und damit „Anfrageabbruch“. Das erscheint mir am logischsten.

Ich schaue, ob ich das heute Abend schaffen kann. Wenn nicht, dann nächste Woche, ich werde eine Woche lang abwesend sein

Ich melde mich

ok, halt mich auf dem Laufenden! Wenn du nicht weiterkommst, kann ich dir helfen

Kleinen Bug bei der Zwave-Integration zu melden, ich habe pro Raum ein Zwave-Gerät, um jeden Heizkörper zu steuern. In zwavejs-ui habe ich daher den Namen als „Heizkörper“ und den Standort als Raum angegeben.

Allerdings sehe ich in Gladys nur Heizkörper (normal), aber es ist schwierig zu erkennen, welcher welcher ist. Vielleicht könnte man den Standort an einer Stelle hinzufügen, damit man die Geräte unterscheiden kann? Aber ich habe Angst, dass es zu einer Doppelfunktion mit dem Raum in Gladys kommt. Wie siehst du das, @pierre-gilles?

Zum eigentlichen Bug: Wenn ich versuche, meine Heizkörper zu Gladys hinzuzufügen, erhalte ich eine Fehlermeldung

Es gibt einen Konflikt mit dem selector, der nicht eindeutig ist.

Beim Durchsehen des Codes habe ich keinen Selector in der Zwavejs-UI-Integration gefunden, daher denke ich, dass der Selector aus dem name gebildet wird, dank dieser Funktion.
image

Ich versuche, etwas zur Korrektur des Selectors vorzuschlagen.

EDIT: Durch das Hinzufügen des Selectors kann ich alle meine Geräte speichern

@pierre-gilles, der PR

Gut gesehen @Will_71! Ich habe dir einen Kommentar auf der PR hinterlassen :slight_smile:

Ich denke, wir müssen die gleiche Arbeit am Feature-Selector durchführen, sonst wird es dort auch hängen bleiben, glaube ich.

Okay, ich kümmere mich darum, sobald es geht.

@pierre-gilles, ich habe die Änderungen am Pull Request vorgenommen.

Danke @Will_71! Für mich ist das okay, es ist in master gemerged und wird in der nächsten Version von Gladys enthalten sein :slight_smile:

Ok​:+1:.
Ich warte dann auf den nächsten PR von @Sescandell, denn das, was er gemacht hat, wird es mir ermöglichen, ein neues Gerät hinzuzufügen.

Ich bereite auch gerade einen PR vor, um den « Standort » jedes Geräts zu integrieren

Hallo,

Ich habe den PR aktualisiert: Rebase auf master, um die letzten Änderungen von @Will_71 zu übernehmen und die Code-Coverage-Probleme zu beheben.

PR für die Dokumentation ist hier verfügbar: Update Zwavejs UI doc by sescandell · Pull Request #255 · GladysAssistant/v4-website (github.com)

Jetzt bist du dran @pierre-gilles :wink:

@pierre-gilles, der PR für den Zusatz des « Orts »

@Sescandell Danke für das PR zur Temperatur + Leistung, das sieht für mich sehr gut aus :slight_smile: Ich habe gemerged und es wird in der nächsten Version von Gladys enthalten sein! Ich werde die Website mergen, sobald die Veröffentlichung live ist

@Will_71 Dein PR ist auch für mich in Ordnung, allerdings gibt es dadurch einen kleinen Konflikt mit dem PR von @Sescandell, das ich gerade vorher gemerged habe, kannst du das bitte prüfen?

Ich versuche, um Mittag während meiner Mittagspause zu schauen.

@pierre-gilles Alles klar, ich habe den Konflikt gelöst