[JacoPad] Arduino #46

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

[JacoPad] Arduino

# [JacoPad] Arduino
Adrien added 11 commits 2024-04-23 17:26:24 +00:00
Adrien added the
JacoPad
Hardware
labels 2024-04-23 17:27:39 +00:00
Adrien added 1 commit 2024-04-23 17:36:22 +00:00
Adrien added 1 commit 2024-04-23 17:38:27 +00:00
Adrien added 1 commit 2024-04-23 17:42:05 +00:00
Adrien added 1 commit 2024-04-23 17:48:56 +00:00
Adrien added 1 commit 2024-04-23 17:52:29 +00:00
benvii requested changes 2024-04-24 09:42:59 +00:00
benvii left a comment
Owner

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" ...
benvii was assigned by HS-157 2024-04-26 20:14:07 +00:00
Adrien added 1 commit 2024-05-04 21:46:24 +00:00
Adrien added 1 commit 2024-05-04 22:04:21 +00:00
Adrien added 1 commit 2024-05-05 09:52:44 +00:00
Adrien added 1 commit 2024-05-05 10:09:12 +00:00
Adrien added 1 commit 2024-05-05 14:18:47 +00:00
Adrien added 1 commit 2024-05-05 19:58:57 +00:00
Adrien added 1 commit 2024-05-06 17:19:13 +00:00
benvii added 1 commit 2024-05-11 14:42:33 +00:00
Adrien added 1 commit 2024-05-11 15:28:43 +00:00
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.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u jacopad:Adrien-jacopad
git checkout 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.