feature/servo_motor_component #17

Merged
Mathis merged 13 commits from feature/servo_motor_component into develop 2024-01-19 15:46:01 +00:00
Member
No description provided.
Mathis added the
Kind/Feature
label 2024-01-19 08:49:41 +00:00
Clement was assigned by Mathis 2024-01-19 08:49:41 +00:00
Gildas was assigned by Mathis 2024-01-19 08:49:41 +00:00
Nicolas was assigned by Mathis 2024-01-19 08:49:41 +00:00
Mathis added 4 commits 2024-01-19 08:49:42 +00:00
Clement requested changes 2024-01-19 10:14:15 +00:00
@ -0,0 +13,4 @@
return this->myservo.read(this->PIN);
}
void ServoMotorComponent::setCurrentPosition() {
Owner

???

???
@ -0,0 +17,4 @@
this->currentPosition = this->myservo.read(this->PIN);
}
void ServoMotorComponent::setDesiredPosition(Position desiredPosition) {
Owner

Nom des variables pas claire

Nom des variables pas claire
@ -0,0 +39,4 @@
|| millis() - this->lastUpTime <= this->updatePeriod) return;
if (this->currentPosition > this->desiredposition) {
this->currentPosition--;
Owner

Limite faire en sorte de pouvoir configurer le pas dans le constructeur (avec une valeur par défaut)
en double ou float

Limite faire en sorte de pouvoir configurer le pas dans le constructeur (avec une valeur par défaut) en double ou float
@ -0,0 +44,4 @@
if (this->currentPosition < this->desiredposition) {
this->currentPosition++;
}
this->lastUpTime = millis();
Owner

typo ctrl C

typo ctrl C
Clement marked this conversation as resolved
@ -0,0 +49,4 @@
}
void ServoMotorComponent::setPin(int PIN) {
Owner

pas besoins car déja fait dans constructeur

pas besoins car déja fait dans constructeur
Mathis marked this conversation as resolved
@ -0,0 +9,4 @@
RIGHT
};
class ServoMotorComponent
Owner

il n'y a pas d'interface

il n'y a pas d'interface
Mathis marked this conversation as resolved
@ -0,0 +13,4 @@
{
public:
ServoMotorComponent(int PIN, unsigned long updatePeriod);
~ServoMotorComponent() = default;
Owner

tu peut viré le destructeur

tu peut viré le destructeur
@ -0,0 +15,4 @@
ServoMotorComponent(int PIN, unsigned long updatePeriod);
~ServoMotorComponent() = default;
int getCurrentPosition();
void setCurrentPosition();
Owner

met des commentaire doxygen sur tes fonction

met des commentaire doxygen sur tes fonction
@ -2,28 +2,17 @@
#include "Arduino.h"
Owner

vire tes changement dans Program.CPP avant de push

vire tes changement dans Program.CPP avant de push
Mathis added 1 commit 2024-01-19 10:35:31 +00:00
Mathis added 1 commit 2024-01-19 15:37:43 +00:00
Nicolas approved these changes 2024-01-19 15:40:56 +00:00
Mathis added 7 commits 2024-01-19 15:43:49 +00:00
Mathis merged commit c498489cbb into develop 2024-01-19 15:46:01 +00:00
Sign in to join this conversation.
No description provided.