Anzeige der Funktionsnamen statt der Gerätenamen für bestimmte Integrationen!

[pierre-gilles]

Hallo zusammen!

Einige von euch haben sich beschwert, dass sie es nicht schaffen, bestimmte Geräte in den Select-Boxen von Gladys von anderen zu unterscheiden. (@Terdious, @Romuald_Pochet)

Ich habe diesen Fall zum Beispiel notiert:

Es stimmt, dass bei einigen Integrationen (Z-Wave, Zigbee, MQTT und Sonoff) manchmal mehrere Funktionen desselben Typs auf einem Gerät vorhanden sein können, und es ist daher derzeit schwer lesbar!

Allerdings ist das nicht bei allen Integrationen der Fall:

Im Gegensatz dazu sind bei anderen Integrationen, Philips Hue zum Beispiel, der Name des Geräts interessant, aber der Name der Funktion kann nicht geändert werden und hat nicht viel Interesse. (Im Fall von Philips Hue sind die Philips Hue Lampen ziemlich einfach: Ein/Aus, Farbe und Helligkeit und das war’s)

Ich habe mich mit dem Problem beschäftigt, und da es unmöglich ist, jeden zufrieden zu stellen, denke ich, dass ich eine unterschiedliche Anzeige je nach Integration machen werde, wie wir es bei der Ansicht « Gerät des Raums » auf dem Dashboard gemacht haben.

Für den Moment bin ich von folgendem ausgegangen:

Für die Integrationen:

• mqtt
• zigbee2mqtt
• zwave
• ewelink
• tasmota

Edit:

• bluetooth

Wir werden den Namen der Funktion anzeigen, und für die anderen den Namen des Geräts!

Was haltet ihr davon?

Fehlen Integrationen in dieser Liste?

[GBoulvin]

Super!
Das wird uns die Arbeit enorm erleichtern!

[pierre-gilles]

Ich füge eine weitere Frage hinzu: Ist es wirklich sinnvoll, in Klammern den Typ der Funktion anzugeben, wo vielleicht der Name ausreicht, was dann so aussehen würde:

[GBoulvin]

Mal sehen, aber ich würde es so lassen (zumindest das entsprechende Symbol).

[pierre-gilles]

Hier geht es nur um den Text, und es gibt kein Icon, weil es um die Ansichten „Trigger Gerätezustandsänderung“ geht

Für mich ist es, basierend auf meinen Tests, extrem redundant, wenn man die Funktion in dieser Ansicht anzeigt. Das wirkt schwerfällig in der UI.

[pierre-gilles]

Der PR ist auf Github:

For some integrations, display the device feature name and not the device name in the UI (#1322)

[Terdious]

Hallo @pierre-gilles,

Ich möchte gerne aufgreifen:

@pierre-gilles

Ich habe mich mit dem Problem beschäftigt, und da es unmöglich ist, jeden zufriedenzustellen, denke ich, dass eine unterschiedliche Anzeige je nach Integration sinnvoll ist, so wie wir es bei der Ansicht „Gerät im Raum“ im Dashboard gemacht haben.

Ich weiß nicht, wann das gemacht wurde, aber bei mir hat es nie funktioniert. Seltsam, dass niemand anderes das gemeldet hat. Bei mir ist das schon mehrmals passiert, aber es ist wohl unbemerkt geblieben ^^ Kein Problem ^^ Ich dachte, es wäre in Arbeit und wieder einmal keine GitHub-Issue!! Ein Beispiel (aber es ist dasselbe bei MQTT und anderen):

Gerät Tasmota:

Dashboard-Bearbeitung:

Dashboard:

Aus diesem Grund wäre es gut, gleichzeitig am Dashboard und der Dashboard-Bearbeitung zu arbeiten

@pierre-gilles

Wir werden den Namen der Funktion anzeigen, und für die anderen den Namen des Geräts!

Was haltet ihr davon?

Aber das macht alles komplizierter, warum nicht für Integrationen wie Philips Hue in der Datenbank im Feld « Name » die Kategorie der Funktion eintragen, wenn es das ist? Im Grunde genommen möchte man bei jeder Integration (AMHA) immer den Namen der Funktion anzeigen. Wenn man den Namen der Philips Hue-Funktion nicht ändern kann, ist das egal, das ist im Code gemacht und man kann ihm den Namen der Kategorie und des Typs geben, wenn man möchte. Und das vermeidet es, spezifische Lösungen zu benötigen, während es einfach bleibt, oder?
Das erinnert mich übrigens daran, dass… warum kann man diesen Namen überhaupt nicht ändern? Wir sind uns doch einig, dass dieser Name auf der Entwicklerseite nie verwendet wird (und zum Glück). Das würde vieles vereinfachen.

[pierre-gilles]

@Terdious

Ich weiß nicht, wann das gemacht wurde, aber für mich hat es nie funktioniert, seltsam, dass es niemand anderes gemeldet hat

Auf dem Dashboard erinnere ich mich, dass du mich nur für MQTT-Geräte gefragt hast, also habe ich den Fix für MQTT-Geräte gemacht Kannst du bestätigen, dass der Fix für MQTT-Geräte auf dem Gerätebox der Raum funktioniert? Bei mir tut er es.

Entschuldigung für deine Rückmeldungen, ich lese viele Nachrichten pro Tag, wenn es keine schriftliche Anfrage gibt und sie im Backlog (GitHub, wenn es ein Bug ist, Forum, wenn es eine Funktion ist) steht, wird es mit 100%iger Sicherheit nicht gemacht ^^

Daher ist es normal, dass Tasmota derzeit so angezeigt wird.

@Terdious

Daher wäre es in diesem Sinne gut, wieder zum Dashboard und zur Dashboard-Bearbeitung zurückzukehren

Ja, das ist erledigt, ich habe die gleiche Liste der Integrationen auf die beiden Ansichten angewendet

@Terdious

Eigentlich, egal welche Integration AMHA, wir möchten immer den Namen der Funktion anzeigen.

Warum die Integrationen komplizierter machen, die einfach sind?

Das ist nicht fair, es würde die einfachen Integrationen aufgrund der anderen komplexen Integrationen kompliziert machen, das ist ein Absenken auf das niedrigste Niveau ^^

Für mich sind MQTT / Zigbee2Mqtt / Tasmota Integrationen für Entwickler und nicht für die breite Masse. Sie haben eine sehr spezielle Funktionsweise, die nicht verallgemeinerbar ist.

Die breite Masse, die Gladys verwendet, wird anschließen:

• Philips Hue
• Sonos
• TP-Link-Steckdosen

Und sie werden erwarten, dass Gladys wie ein Google Home / Apple HomeKit / Alexa funktioniert.

Diese Plattformen zeigen ihren Nutzern nicht einmal die Konzepte von deviceFeatures an, in Google Home / Home Kit / Alexa zählt nur der Name des Geräts, und ich denke, wir sollten dasselbe für unsere « Public » Integrationen tun.

[pierre-gilles]

Edit: Ich habe die Bluetooth-Integration zur Liste hinzugefügt

[Terdious]

@pierre-gilles

Auf dem Dashboard erinnere ich mich, du hattest mich nur für MQTT-Geräte gefragt, also habe ich den Fix für MQTT-Geräte gemacht Kannst du bestätigen, dass der Fix für MQTT-Geräte auf dem Geräte-Box der Raumfunktion funktioniert? Bei mir tut er es jedenfalls.

Ich bestätige, dass wir auf dem Dashboard den Namen abrufen, aber auf der Bearbeitungsseite in der Auswahl gilt dies nicht für MQTT. Es erscheint als Zigbee2Mqtt und Tasmota:

Die vier Schalter oben heißen « Öffnung […] », « Schließung […] », usw.

@pierre-gilles

Entschuldigung für deine Rückmeldungen, ich lese viele Nachrichten pro Tag, wenn es keine schriftliche Anforderung gibt und im Backlog (GitHub, wenn es ein Bug ist, Forum, wenn es ein Feature ist), dann wird es mit 100%iger Sicherheit nicht gemacht ^^

Überhaupt kein Problem!! Ich verstehe das vollkommen!!

[pierre-gilles]

@Terdious

Ich bestätige, dass im Dashboard der Name korrekt angezeigt wird, aber auf der Bearbeitungsseite wird er in der Auswahl nicht angezeigt, auch nicht bei MQTT. Es wird als Zigbee2Mqtt und Tasmota angezeigt:

Alles ist also normal

Ich werde bald den PR mergen, das wird in der nächsten Gladys-Version veröffentlicht!

Bearbeitung: Es ist in master gemergt!

[lmilcent]

Ich habe von Anfang an geschummelt, indem ich zuerst die Zigbee2Mqtt-Schnittstelle verwendet habe, um die Geräte umzubenennen, sie dann zu „scannen“ und mit einem sinnvollen Namen in Gladys zu integrieren
Tolle Idee übrigens, das wird viel einfacher

[Romuald_Pochet]

Hallo,

Ich habe gerade den PR erhalten. Ich habe versucht, meine Szene zu aktualisieren, aber der Gerätename wird nicht mehr angezeigt…

Wenn ich den Gerätenamen hinzufüge, ist alles perfekt für mich (abgesehen von den Z-Wave-Feature-Namen, die ich in ein Wörterbuch setzen müsste)

PS: Ich habe die Z-Wave-Node-ID in den Gerätenamen gesetzt, um ihn leichter wiederzufinden, das wird verschwinden

[pierre-gilles]

@Romuald_Pochet

Ich habe gerade den PR erhalten. Ich habe versucht, meine Szene zu aktualisieren, aber der Gerätename wird nicht mehr angezeigt…

Ah, genau das macht der PR eigentlich Aber vielleicht passt das nicht so gut zu Z-Wave… Das können wir ändern.

Hier ist, was ich geändert habe:

Im Großen und Ganzen wird für Geräte der Integrationen:

• MQTT
• Z-Wave
• Zigbee2mqtt
• Ewelink
• Tasmota

nur noch der Name des DeviceFeature angezeigt, der Gerätename wird nirgends mehr angezeigt.

Gibt es damit Probleme mit Z-Wave?

[Romuald_Pochet]

Angenommen, ich habe mehrere Geräte mit einer Temperatur-Funktion (im z-Wave ist das der Name der zugehörigen Klasse, im Zigbee das „expose“). Dann wird „Temperatur“ also mehrmals vorkommen, oder?

Was ich gemacht habe:

const getDeviceFeatureName = (dictionnary, device, deviceFeature) => {
  const deviceService = get(device, 'service.name');
  const featureDescription = get(dictionnary, `deviceFeatureCategory.${deviceFeature.category}.${deviceFeature.type}`);
  if (deviceService && DISPLAY_FEATURE_NAME_FOR_THOSE_SERVICES[deviceService]) {
    return `${device.name} (${deviceFeature.name})`; 
  }
  return `${device.name} (${featureDescription})`;
};

[pierre-gilles]

@Romuald_Pochet

Ce que j’ai fait:

Ja, aber genau dafür war die Entscheidung, den device.name nicht mehr zu verwenden.

Wenn es darum geht, kann ich die Änderung bei Z-Wave zurücknehmen, um es wieder wie früher zu machen.

Das Fundament der Gladys-Philosophie ist es, ein Produkt zu schaffen, das von jedermann genutzt werden kann, auch von nicht-technischem Publikum.

Die Frage wäre also: Wie können wir es schaffen, dass die Z-Wave-Geräte einen „menschlichen“ Namen haben, der lesbar ist und es ermöglicht, das Gerät leicht zu identifizieren.

Denn von den drei Vorschlägen, die ich für Z-Wave sehe (derzeit, der PR, den ich gestern gemacht habe, und deinem Vorschlag), erscheint mir dein Vorschlag am Ende der sauberste (also PR von gestern zurücknehmen).

[Romuald_Pochet]

Völlig einverstanden mit der Philosophie

Wenn wir das Gerät (Name oder Ähnliches) nicht in der Dropdown-Liste anzeigen, müssen wir sicherstellen, dass die Bezeichnung der Funktion eindeutig ist. Ist das vorstellbar? Wenn ich zwei Temperatursensoren habe (Gerät + 1 Feature Temperatur), kann ich mein Feature nicht einfach ‹ Temperatur › nennen, sondern muss es z. B. ‹ Temperatur Sensor Toto › oder ‹ Temperatur Wohnzimmer › nennen.

Das ist nicht mit einer bestimmten Integration verbunden. Bei MQTT müssen wir einen Namen angeben, daher ist das Problem geringer, außer dass jeder Funktion ein eindeutiger Name zugewiesen werden muss. Wir könnten von diesem Prinzip ausgehen, der Benutzer wählt einen Namen für die Funktion.

[pierre-gilles]

@Romuald_Pochet

Wenn wir das Gerät (Name oder andere) nicht in der Dropdown-Liste angeben, müssen wir sicherstellen, dass die Bezeichnung der Funktion eindeutig ist. Ist das möglich?

Ganz genau, es ist tatsächlich etwas umständlich, den Benutzer zwingen zu müssen, jede Funktion jeder Integration zu benennen.

Bei MQTT verstehe ich das ja (das ist manuell), aber am Ende wird es bei Integrationen wie Z-Wave schnell ein Albtraum… Vor allem, weil bei kommerziellen Smart-Home-Lösungen niemand das macht.

Entschuldige @Terdious, aber ich denke, ich werde die PR für die meisten Integrationen zurücksetzen (ich denke, ich lasse nur MQTT), wir haben zu wenig Informationen, ich habe das etwas zu überstürzt und ohne die anderen Community-Mitglieder zu konsultieren, die unterschiedliche Setups haben, gemerged.

@Romuald_Pochet im Fall von Z-Wave ist es in 99% der Fälle in Ordnung, eine Kombination aus « Gerätename + (Funktionsname aus den Übersetzungen) » anzuzeigen. Ich denke, es sind wirklich nur die 1% der seltsamen Geräte, die Probleme bereiten (die, über die wir im Z-Wave-Thread sprechen, siehe https://community.gladysassistant.com/t/v4-integration-z-wave/6057/53?u=pierre-gilles), und bei denen sollten wir versuchen, sie in mehrere Geräte aufzuteilen, anstatt unsere gesamte Vorgehensweise für diese 1% zu ändern.

Ich denke, um diese Diskussion fortzusetzen, müssen wir mehr Informationen sammeln.

Erstens, auf Gladys-Ebene:

• Eine Liste der Integrationen in Gladys erstellen und für jede Integration auflisten, wie die Benennung derzeit funktioniert (Gerätename und Funktionsname des Geräts)
• Die problematischen Fälle identifizieren und diese Fälle individuell verstehen: Macht es wirklich Sinn, ein einziges Gerät mit 200 Funktionen zu haben? Sollte das Gerät nicht besser aufgeteilt werden?

Zweitens, außerhalb von Gladys:

• Jedes kommerzielle Smart-Home-System ansehen: Google Home, Alexa, Apple Homekit (ich betone das kommerzielle: Home Assistant, Domoticz sind keine Inspirationen, ich sage nicht, dass diese Systeme nicht gut sind, aber sie sind für ein anderes Publikum bestimmt, also ist das nicht das, woran wir uns hier orientieren wollen).

Wenn jemand diese kleine Umfrage durchführen möchte, ist er herzlich willkommen Ich kann Zeit dafür investieren, aber ich muss das mit allen anderen geforderten Funktionen priorisieren (ist das prioritätswürdiger als die grafische Ansicht? als die Alexa-Integration? Das liegt an euch, mir zu sagen).

[pierre-gilles]

Ich habe gerade die Änderung in diesem PR zurückgesetzt: User patch by Jean-PhilippeD · Pull Request #132 · GladysAssistant/Gladys · GitHub

Die Anzeige des deviceFeature.name bleibt nur für MQTT wie zuvor.

[pierre-gilles]

Ich habe eine kleine Untersuchung über andere öffentliche Smart-Home-Systeme durchgeführt:

Apple Homekit

Jedes Gerät hat einen Namen (äquivalent zu deviceName), und nur dieser Name wird angezeigt.

Es gibt keine Vorstellung von deviceFeature, zumindest wird dies nicht öffentlich zugänglich gemacht, man sieht nur ein Gerät mit « capabilities », die nicht benannt sind:

Google Home

Gleiches Prinzip wie Apple Homekit, jedes Gerät hat einen Namen und einen Gerätetyp, und jedes Gerät hat « Eigenschaften » (eine Feature).

In der UI wird nur der Gerätename angezeigt:

Das, was man im Code der Google Home-Integration sieht, sieht ein Gerät so aus:

{
   id: 'UN_ID_UNIQUE',
    type: 'action.devices.types.LIGHT',
    traits: ['action.devices.traits.OnOff'],
    name: {
      name: 'Wohnzimmerlampe',
    },
    deviceInfo: {
      model: 'XXXX',
    },
    roomHint: 'Wohnzimmer',
    willReportState: true,
}

Es gibt keine Vorstellung von deviceFeature name, jedes physische Gerät hat einfach einen Namen.

Amazon Alexa

Die Funktionsweise ist Google Home recht ähnlich, jedes Gerät wird durch seinen Namen und eine Liste von « capabilities » dargestellt. Die capabilities scheinen weder benannt noch in der Oberfläche angezeigt zu werden.

Daher frage ich mich: Wie gehen Leute mit komplexen Geräten wie Mehrfachsteckdosen um, um diese auf diesen Plattformen zu verwalten?

Je länger ich darüber nachdenke, desto weniger überzeugt bin ich, dass der feature.name einen Sinn ergibt…