[JacoPad] Arduino #46

Open
Adrien wants to merge 25 commits from Adrien/JacoBot:jacopad into main
Member

[JacoPad] Arduino

# [JacoPad] Arduino
Author:    Linux_Hat <adrien@d4x.fr>
This reverts commit 97dd28a88a.

	deleted:    jacopad/arduino/README.md
	deleted:    jacopad/arduino/main.cpp
	deleted:    jacopad/arduino/token.cpp
	deleted:    jacopad/arduino/token.h
This reverts commit ea3a44c024.

	modified:   jacobot/arduino/Car.cpp
	modified:   jacobot/arduino/MotorEncoder.cpp
	modified:   jacobot/arduino/main.cpp
	deleted:    jacopad/arduino/.gitkeep
benvii left a comment

Good job but need to add an enum to represent each token type and return the enum value.

Good job but need to add an enum to represent each token type and return the enum value.
@ -0,0 +3,4 @@
// Token class declaration
class Token {
private:
int m_pin; // Pin number connected to the sensor
Owner

Add that this pin will measure the resistance.

Add that this pin will measure the resistance.
Adrien marked this conversation as resolved
@ -0,0 +11,4 @@
public:
// Constructor declaration
Token(int pin, float base, float limits[3], float min, float max);
Owner
  • Add doxygen doc and explain the format of limits array, like this :
 * @param limits array meaning, for those values : {1000.0, 4000.0, 7500.0}
                     < 1000 Ω                 <  4000 Ω                     <  7500 Ω
      Token 1                   Token 2                  Token 3             Token 4
  • You will need to add a table with the list of instruction named associated to each type of token or make a enum then return the number of the enum.

  • Document base, indicate that this this the constant resistance value use in the tension divider (outside of the token)

- [ ] Add doxygen doc and explain the format of limits array, like this : ``` * @param limits array meaning, for those values : {1000.0, 4000.0, 7500.0} < 1000 Ω < 4000 Ω < 7500 Ω Token 1 Token 2 Token 3 Token 4 ``` - [ ] You will need to add a table with the list of instruction named associated to each type of token or make a enum then return the number of the enum. - [ ] Document base, indicate that this this the constant resistance value use in the tension divider (outside of the token)
Adrien marked this conversation as resolved
@ -0,0 +17,4 @@
float read();
// Method to determine and return the type of token based on the sensor value
int type();
Owner

Type should return the instruction name "FORWARD", "LEFT, "RIGHT" ...

Type should return the instruction name "FORWARD", "LEFT, "RIGHT" ...
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u jacopad:Adrien-jacopad
git switch Adrien-jacopad
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
MDL29/JacoBot!46
No description provided.