francispoisson / hilo

Hilo integration for Home Assistant
The Unlicense
26 stars 8 forks source link

Ampoules non détectées #27

Closed vgibara closed 2 years ago

vgibara commented 2 years ago

Salut !

Superbe boulot avec hilo ! Bravo !

Une question toutefois. J'ai des thermostats et des prises intelligentes, que je peux voir dans HA. Mais j'ai aussi des ampoules que je ne vois pas. Est-ce normal ? Je vois pourtant le fichier light.py mais je crois que ce n'est que pour les dimmer. Je me trompe ?

Comment puis-je ajouter le support pour les ampoules ?

Merci !!

francispoisson commented 2 years ago

Une autre personne a proposé une modification dans les "pull request", tu peux tester ça et confirmer que ça fonctionne, je n'ai pas de lumières ici.

Le ven. 17 sept. 2021, à 13 h 37, Vincent Gibara @.***> a écrit :

Salut !

Superbe boulot avec hilo ! Bravo !

Une question toutefois. J'ai des thermostats et des prises intelligentes, que je peux voir dans HA. Mais j'ai aussi des ampoules que je ne vois pas. Est-ce normal ? Je vois pourtant le fichier light.py mais je crois que ce n'est que pour les dimmer. Je me trompe ?

Comment puis-je ajouter le support pour les ampoules ?

Merci !!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/francispoisson/hilo/issues/27, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRNQSFOPYQBXKPNCC7PDRDUCN4EDANCNFSM5EIAT2DQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

vgibara commented 2 years ago

Effectivement. Le pull gère les Dimmer (interrupteurs), mais pas les ampoules. J'ai modifié temporairement la ligne 15 pour:

if(hass.data[DOMAIN].d[i].deviceType == 'LightDimmer' or hass.data[DOMAIN].d[i].deviceType == 'WhiteBulb' or hass.data[DOMAIN].d[i].deviceType == 'ColorBulb'):

Ça fonctionne très bien. Toutefois, je cherche encore comment gérer les couleurs... je pourrais faire un vrai pull request et faire les changements pour un éventuel commit. Quelqu'un peut m'aider avec la gestion des couleurs ?

francispoisson commented 2 years ago

Pour le colorbulb, où en es-tu?

Peux-tu tester ce code dans ton environnement de programmation pour voir quelles sont les attributs des ampoules de couleur en changeant le "0" pour correspondre à l'une de tes ampoules?

from hiloapi import *

def main(): h = Hilo('username', 'password')

h.update()

print(h.d[0].name)
print(h.d[0].supportedAttributes)
print(h.d[0].settableAttributes)

if name == "main": main()

vgibara commented 2 years ago

J'ai peu avancé. J'en étais rendu à setter des valeurs pour ces variables justement. ColorMode, Hue, Saturation et Intensity. Mais j'ai pas fini l ;)

Lumière Cuisine 1 OnOff, Intensity, Disconnected, Hue, Saturation, ColorTemperature OnOff, Intensity, ColorMode, Hue, Saturation, ColorTemperature

Le mar. 5 oct. 2021, à 08 h 54, francispoisson @.***> a écrit :

Pour le colorbulb, où en es-tu?

Peux-tu tester ce code dans ton environnement de programmation pour voir quelles sont les attributs des ampoules de couleur?

`from hiloapi import *

def main(): h = Hilo('username', 'password')

h.update()

print(h.d[0].name)

print(h.d[0].supportedAttributes)

print(h.d[0].settableAttributes)

if name == "main": main()`

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/francispoisson/hilo/issues/27#issuecomment-934384604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFYJKGDG7OXB37EKR6CTSTUFLYPLANCNFSM5EIAT2DQ .

francispoisson commented 2 years ago

Pour les ampoules, ça semble ok pour la détection. Je vais créer une nouvelle issue pour l'attribut "couleur".