zigbeefordomoticz / Domoticz-Zigbee

Zigbee plugin for Domoticz. Allow to connect various zigbee controllers like Zigate but also Texas Instrument CC2531, CC13x2, CC26x2 ; Silicon-Labs; deConz based chipset to be connected to Domoticz
GNU General Public License v3.0
99 stars 43 forks source link

EndPoints numbering and naming convention #125

Closed pipiche38 closed 6 years ago

pipiche38 commented 6 years ago

4/ J'ai commencé à ajouter le numéro des EndPoints pour le nommage par défaut des "devices Domoticz", je trouve cela pratique (surtout quand j’avais tout en double) : à continuer ou à supprimer.

cf: https://easydomoticz.com/forum/viewtopic.php?f=8&t=5254&start=480#p57711

thiklop commented 6 years ago

Je ne comprends pas ce que tu as fait. J'ai des capteurs Température/Humidité/Pression, est-ce que cela signifie qu'il faut créer trois lignes différentes ?

pipiche38 commented 6 years ago

Il faudrait que @cldfr commente

cldfr commented 6 years ago

Mon commentaire 4/ faisait référence à cette modification :

Code origine :

if t=="Temp" : # Detecteur temp self.ListOfDevices[DeviceID]['Status']="inDB" Domoticz.Device(DeviceID=str(DeviceID),Name=str(t) + " - " + str(DeviceID), Unit=FreeUnit(self), TypeName="Temperature", Options={"Zigate":str(self.ListOfDevices[DeviceID]), "TypeName":t}).Create()

Le code proposé sur le forum (dans ma modification) : if t=="Temp" : # Detecteur temp self.ListOfDevices[DeviceID]['Status']="inDB" Domoticz.Device(DeviceID=str(DeviceID),Name=str(t) + " - " + str(DeviceID) + " - " + str(Ep), Unit=FreeUnit(self), TypeName="Temperature", Options={"Zigate":str(self.ListOfDevices[DeviceID]), "TypeName":t}).Create() # Ajout de l'EP pour différentier dans Domoticz un même type de capteur sur deux EP diffents d'un même deviceD

J'ai juste ajouté : + " - " + str(Ep) Ce qui permet d'avoir dans le nom proposé le nom du EndPoint.

J'ai fait cette modification à 4 endroits seulement : Temp, Lux, Switch, Door (périphérique Domoticz que j'ai ajouté pour ce dernier), d’où mon commentaire: à continuer ou à supprimer

Par ailleurs, je pense qu'il serait bien d'enlever les espaces dans les noms proposés, pour éviter d'avoir des espaces dans le noms des fichiers scripts LUA .

Actuellement : "XXXX - Temp" par "XXXX-Temp" ou "XXXX - Temp - 01" (avec ma modification) par "XXXX-Temp-01"

Par ailleurs, il n'y a pas de capteur humidité et pression dans le composant ZigBee que j'ai utilisé (juste Temp et Lux).

Pour des capteurs "Température/Humidité/Pression" ça ne change rien.

Par exemple : la modification proposées permet à un composant ZigBee qui proposerait deux switchs, l'un sur l'EP 1 l'autre sur l'EP 2 d'avoir deux noms différents.

XXXX-Switch-01 et XXXX-Switch-02

Sinon, actuellement on à 2 fois : XXXX-Switch (ce qui n'est pas pratique pour savoir qui est qui :-))

pipiche38 commented 6 years ago

Je trouve cela plutôt pas mal! Les 2 points me semblent une excellente suggestion. @cldfr peux-tu te charger de faire cela dans la branche developement sur l'ensemble des Devices Domoticz créer ? et soumettre un PR ?

cldfr commented 6 years ago

Oui, mais en fin de semaine prochaine...

pipiche38 commented 6 years ago

Merged with 7ea6deb based on #154