search

ClubCedille / jardiniot

Système pour des buckets autonomes servant à faire pousser des plantes
GNU General Public License v3.0
13 stars 11 forks source link

Mise à jour des différents senseurs avec les nouvelles valeurs reçues #50

Closed AXDOOMER closed 7 years ago

AXDOOMER commented 7 years ago

Voici un commit très simple qui ajoute le code qui écrit les valeurs reçues au Arduino dans les bonnes pins. Il s'agit d'écrire les bonnes valeurs dans les pins en analogue, ces pins doivent supporter les PWM.

Le code a été testé au préalable dans un programme de test qui peut être retrouvé dans le commit référencé ci-dessous. Il fonctionne avec certitude, cependant je pense que les derniers commits qui ont été fait dans le repo du projet ont cassé de quoi, car le Arduino crash. Pour être clair: C'est pas mon code qui le fait crasher, car il crashait avant mes changements. Je n'ai pas inclut tous mes changements au code pour que ça soit clair, j'ai juste mis ce qui était pertinent à faire marcher les DEL et les fans.

Ce commit résout le issue https://github.com/ClubCedille/jardiniot/issues/49.

Après ce commit, le projet quitte le stade de prototypage, cependant on devra trouver ce qui fait crasher de Arduino. Description du problème: Quand le Arduino se connecte à MQTT box, on reçoit juste "hello world" à l'intérieur du topic qui reçoit les données (celui qui débute par status_), puis mosquitto affiche dans le terminal que le ESP s'est déconnecté à cause d'un erreur.

mikefaille commented 7 years ago

Je peux donner mon commentaire Samedi. Cheers

aubguillemette commented 7 years ago

Après ce commit, le projet quitte le stade de prototypage, cependant on devra trouver ce qui fait crasher de Arduino.

Selon moi, tant que ça crash, c'est encore en stade de prototypage.

Hormis ça, le code a l'air pas pire. Mon seul commentaire serait de pas mixer les commentaires français et anglais (j'sais que tu les as écrit en français et que ceux qui sont en anglais ont probablement été écrits par quelqu'un d'autre), mais hormis ça j'pense que c'est beau. Si @mikefaille a pas d'autre chose à dire, c'est O.K. pour moi.

AXDOOMER commented 7 years ago

Oups

AXDOOMER commented 7 years ago

C'était supposé quitter le stade de prototypage, mais après ce commit, le prototype est fonctionnellement complet, car tous les fonctionnalités sont là. Il y a juste ce "bug" très sévère qui fait que le Arduino "crash" après avoir envoyé son "hello world".

Avant de déclarer le prototype complètement terminé, ce bug sera corrigé et je vais aussi standardiser le code, car le style est différent à plusieurs endroits. On devrait aussi choisir si le code doit être en français ou en anglais, ça devra être la même chose dans le code source de l'application web et du serveur.