clubcapra / markhor

🐐 Capra-Markhor is a ROS-based solution for managing and operating Club Capra's second rescue robot. 🐐
9 stars 1 forks source link

Publish flipper target position #31

Closed amDeimos666 closed 2 years ago

benmalenfant commented 2 years ago

Cool to have dans le futur : ce serrait bien de publier la position actuelle du flipper, ça permuterait de voir si on est stuck ou le flipper lag behind

amDeimos666 commented 2 years ago

did you use any other software for formatting the code other than clang-format ?

Tu utilises quoi ? y'as tu des cas spécifiques ou c'est en général ?

lvanasse commented 2 years ago

did you use any other software for formatting the code other than clang-format ?

Tu utilises quoi ? y'as tu des cas spécifiques ou c'est en général ?

Clang-format, what do you use ? And generally we only use clang-format since there's a .clang-format file in the repository, so everybody format the same way. And there's formatting that you made in your PR that I haven't seen before, and I'm simply wondering if that's because I or someone else forgot to do it or that you used something else. In the later case, I'd ask simply to format with clang-format.

amDeimos666 commented 2 years ago

Pour le voltage, Leo a fait des tests et arrive a ses conclusions:

  1. On ne peut lire la tension au borne de chaque batterie parce qu'elles sont branchees en paralleles
  2. On peut mesurer la tension d'une batterie a une barre 2.1 Afficher un message d'alarme lorsque la tension moyenne des batteries est faible
  3. La fonction getBusVoltage() de CTRE, la seule fonction de voleage disponible nous retourne une valeur qui est cap a 16.75
benmalenfant commented 2 years ago

16,75V c'est pas mal bas, on est malheureusement en dehors de leur range 😢 faudra voir c'est quoi la tension des batteries quand elles sont déchargées, si c'est <16,75, on vas pouvoir un low battery warning mais on ne pourra pas avoir le pourcentage des batteries

À part ça, amazing work, thank you😁

benmalenfant commented 2 years ago

Woooww, Alexis la machine!