Nouvelles catégories / nouveaux types dédiés VE

Demande de fonctionnalités
,
41

Bon en espérant que le code pour la conversion convienne, il fonctionne parfaitement. Si on change notre unité de distance « imperial/US » ou « SI », on convertit les unités de distances associées.

image
image246×112 4.36 KB

Je suis donc revenu sur les datas vues de base, j’ai garder les datas supplémentaires vues plus haut dans un autre fichier en « Step 2 ».

image
image426×899 22.2 KB
image
image421×894 22.1 KB

Il me reste les icones à implémenter, le reste est push.
@pierre-gilles, tu peux regarder l’implémentation de conversion si tu le souhaites.

Concernant les états, à votre avis :

Version icônes

Etats

  • Au moins une porte ouverte image / Toutes portes fermée image

  • Au moins une fenêtre ouverte image / Toutes fenêtre fermée image

Commandes => Bouton icône cliquable

  • Déverrouillage (Véhicule verrouillé image / Verrouillage (Véhicule déverrouillé) image

  • Activation alarme (Alarme désactivée) image / Désactivation (Alarme activée) image

Version textuelles

Etats

image
image439×116 2.77 KB

ou
image
image349×114 2.89 KB

Commandes

image
image346×125 3.93 KB
image
image437×126 4.26 KB

42

J’aurais fait l’inverse dans les icônes, ça me semble plus logique. On voit que la porte est ‹ fermée › ou ‹ pas fermée ›, fenêtre ‹ fermée › ou ‹ pas fermée ›.
Non?

43

:sweat_smile:
Totalement d’accord @GBoulvin !! Je me suis seulement emmêlé les pinceaux ^^
J’édite et j’ajoute le reste ^^

1 « J'aime »
44

Ma préférence finalement …

image
image438×349 10.8 KB
image
image439×844 23 KB

45

@pierre-gilles,

Pour faciliter la relecture, j’ai séparé les PR en plusieurs. Elles sont toutes fonctionnelles et peuvent donc être review séparément … ou presque.

1 « J'aime »
46

Peut être un détail pour le moment, mais au niveau des consommations, il faudrait qu’on précise s’il s’agit des celle depuis le départ / depuis la dernière recharge / du dernier trajet.
Car en l’état on ne peut pas le deviner.

Super travail en tout cas !!! :grinning:

1 « J'aime »
47

@guim31,

Non tu as raison, ce n’est pas un détail car pour les datas des API, les types placés ne pourront pas être bougés après coup.

Pour la 1ère étape, il ne s’agira que des valeurs envoyées par l’API, les types sont liés, ensuite c’est au développement des API qu’on pourra spécifier dans le Nom de la feature ce que ça représente. Il n’y a que les spécifiques qui peuvent être liées à de l’interface qui ont un type spécifique. Il y aura ensuite la possibilité d’ajouter des types plus généralistes qui pourront être affiliés à du Custom si on veut faire des calculs (via MQTT fake device) et là ce sera à l’utilisateur de mettre les bons noms.

Dans les captures d’écran que j’ai mis, ce sont des fake_device que j’ai mis pour les tests, je n’ai pas forcément bien répercuté les noms des features.

1 « J'aime »
48

Je t’ai fais un retour sur cette PR :

1 « J'aime »
49

Beau boulot sur la 2ème PR :slight_smile:

Je t’ai fais des retours :

Je t’ai fais des retours :

1 « J'aime »
50

Corrigée … enfin proposition ^^

51

Corrigée !!

52

Corrections faites !! Merci pour ta rigueur (que je n’ai pas …).

Bonne semaine à tous !! Je pars en vacances !! :sweat_smile:

3 « J'aime »
53

Profite bien :sunglasses:

1 « J'aime »
54

Profite bien ! :slight_smile: Je vais regarder tout ça

1 « J'aime »
55

@Terdious

  • Pour la PR n°1, j’ai ajouté un petit commentaire, et il y aussi un commentaire de @S-Axel !
  • Pour la PR n°2, ça me parait bien, mais j’aimerais faire un petit test une fois que la PR n°1 sera mergée.
  • Pour la PR n°3, pareil une fois que tout est mergé j’aimerais faire un test complet pour tester que ça marche bien, mais niveau code ça me parait bien :slight_smile:

Beau boulot ! :clap:

Profite bien de tes vacances, on regarde ça ensemble à ton retour :slight_smile:

3 « J'aime »
56

Merci pour vos retours @S-Axel et @pierre-gilles.
Les correction de la PR1 sont faites.

1 « J'aime »
57

@pierre-gilles ? Tu as pu voir ? Il manque des choses ?

58

Salut :slight_smile: J’étais OFF fin de semaine dernière avec le pont, je reprend Gladys ce matin, je regarde dans la journée !

1 « J'aime »
59

@Terdious Le fait que l’intensité de consigne évolue en incrément de 1 seulement, c’est pratique ou pas ? (Je me rend pas compte des valeurs que ça va être)

1 « J'aime »
60

Salut @pierre-gilles,

Oui, généralement sur de la commande d’intensité comme ça, on n’augmente pas différemment de cette valeur sur l’échelle Ampère. Et sincèrement je ne vois pas d’autre utilisation que pour une voiture ou des batteries solaires à ce jour.

Pour un véhicule par exemple en règle générale on est sur une échelle de 5 à 32A dans la plupart des cas (certaines visiblement vont de 1 à 32A) et en puissance on est donc sur des incréments de 250W.

1 « J'aime »