Retour Gladys V4.0.4 - Dev Color-Picker

Terdious · January 9, 2021, 11:15am

Bonjour,

Tout d’abord, il est important de remercier @AlexTrovato pour son travail et pour cette PR attendue par tous les possesseurs d’ampoule colorée.

Petit retour sur l’utilisation :

1- Lorsqu’on arrive sur le Dashboard, la case au départ est grise et ne contient pas la couleur actuelle. Egalement, si l’on change la couleur, la case prend bien la couleur, mais dès que l’on fait un refresh la case reprend en gris. Problème déjà connu pendant les tests, une PR sera-t-elle prévue ?

2- A l’ouverture du color-picker que ce soit sur le dashboard du navigateur (Chrome) ou sur le téléphone, la fenêtre descend complètement vers le bas, cachant ainsi les cases de la pièce et des features du device. Est-il possible d’éviter ce phénomène de glissement automatique et de ne le faire que si nécessaire (feature en bas de page = remonter la fenêtre de la hauteur du color picker seulement).

Petit GIF pour visualiser les points 1 et 2

A part ça il est bien agréable de pouvoir piloter ça directement depuis Gladys. Vivement la Température et la Luminosité ^^

Spoiler ? ^^

Edit : Le GIF ne fonctionne a priori pas, désolé. J’ai fait plusieurs tentatives, le fiichier upload bien, le lien est ![Color-Picker|1x1](upload://ju27NM0m0AJA7nlX87iZWYIFmBK.gif) mais rien ne s’affiche dans l’aperçu. J’ai fait plusieurs tentatives avec des fichiers de 2 à 4MB mais rien de plus.

Edit-2 : C’est bon, merci @Jean34

Jean34 · January 9, 2021, 10:57am

Héberge le sur casimage où autre ton gif

VonOx · January 9, 2021, 12:12pm

Punaise le dashboard

Terdious · January 9, 2021, 12:31pm

Et encore, il n’y a pas encore tout le matos Netatmo … Ca va être illisible après ça !! Vivement le multi-Dashboard ^^ Après une solution à court terme pourrait-être juste un champ recherche !!

Tlse-vins · January 9, 2021, 5:13pm

Je n’ai pas eu ton problème numéro 1, j’ai essayé plusieurs fois mais tout fonctionne bien de mon côté.

cicoub13 · January 9, 2021, 6:11pm

Pour le problème 1, je pense que suivant l’intervalle de poll, la valeur réelle de l’ampoule n’a pas encore été récupéré quand tu vas sur le Dashboard (juste après avoir ajouté le device).
Tu peux essayer d’ajouter le device et d’attendre que le premier poll survienne pour vérifier ?

Terdious · January 9, 2021, 6:16pm

Aucun soucis là-dessus @cicoub13, les tests ont été fait quelques heures après l’ajout. De plus, une fois la couleur sélectionnée une première fois, elle passe dans la DB en “last_value” dans la table t_device_feature. Donc même si le poll n’avait pas encore eu lieu, au refresh de la page, elle aurait dû garder la couleur sélectionnée auparavant.

Tu as un comportement différent chez toi ?

AlexTrovato · January 10, 2021, 6:52pm

Salut, merci pour les retours.
Ton device avec color picker est sur quel service d’intégration ?
Ce que dit @cicoub13 n’est pas forcément deconnant.

Terdious · January 10, 2021, 7:19pm

Salut,
Tous mes devices avec colorPicker sont sur Philips Hue.

VonOx · January 10, 2021, 9:55pm

@AlexTrovato je suis dans le même cas que @Terdious sur du Philips hue

Goulby · January 11, 2021, 5:53am

De mon côté je testerai ça la semaine prochaine. J’ai une aeotec LED Bulb

pierre-gilles · January 11, 2021, 8:41am

Pour ce point, le poll Philips Hue n’agit que sur le binary on/off pour l’instant car il n’y avait pas la couleur jusque-là, c’est un manquement “normal”, il faut développer la feature

Cf code:

github.com

GladysAssistant/Gladys/blob/master/server/services/philips-hue/lib/light/light.poll.js#L28


  if (!hueApi) {
    throw new NotFoundError(`HUE_API_NOT_FOUND`);
  }
  const state = await hueApi.lights.getLightState(lightId);
  const currentBinaryState = state.on ? 1 : 0;
  const binaryFeature = getDeviceFeature(device, DEVICE_FEATURE_CATEGORIES.LIGHT, DEVICE_FEATURE_TYPES.LIGHT.BINARY);
  // if the value is different from the value we have, save new state
  if (binaryFeature && binaryFeature.last_value !== currentBinaryState) {
    logger.debug(`Polling Philips Hue ${lightId}, new value = ${currentBinaryState}`);
    this.gladys.event.emit(EVENTS.DEVICE.NEW_STATE, {
      device_feature_external_id: `${LIGHT_EXTERNAL_ID_BASE}:${bridgeSerialNumber}:${lightId}:${DEVICE_FEATURE_TYPES.LIGHT.BINARY}`,
      state: currentBinaryState,
    });
  }
}
module.exports = {
  poll,
};

Après, hors poll, il y a quand même un problème je pense !