-
Notifications
You must be signed in to change notification settings - Fork 61
Upgraded EZ to PROS 4.2.1 and to okapi 5.1.0 #309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
|
In testing it seems to work well so far no issues to report yet. |
|
Tested on robot and it appears completely broken at the moment |
|
What does "completely broken" mean? What's not working? The entire robot isn't functioning? New features aren't working? |
|
I have the 4.2.1 version of Ez-Template currently installed on my team robot and had experience the same issue of the robot not functioning with black screen no action being ran. Just a frozen state. The way I made it be able to run was manually copying over all the header and implementation files over to my project. |
| @@ -1,30 +1,11 @@ | |||
| # Dependencies | |||
| /node_modules | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep node_modules in gitignore, can mess with binaries
| __undef_stack = .; | ||
| } > COLD_MEMORY | ||
|
|
||
| /* There are no sections for the stack. This is intentional, since VEXOs sets up a stack before |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unlikely that this caused the error you're seeing if this is true, but still something to point out.
| * | ||
| * \param active | ||
| * sets piston to true | ||
| * \param active |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I keep seeing these, who messed up the params 😭
| #include <unistd.h> | ||
| #endif /* __cplusplus */ | ||
|
|
||
| #define PROS_VERSION_MAJOR 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these necessary? that might also be an issue, depending on if/how pros handles these
Summary:
Upgraded EZ to PROS 4.2.1 and upgraded to okapi 5.1
Motivation:
Allows for the use of AI sensors.
References (optional):
purduesigbots/pros-vsc#262 (comment)
Test Plan:
Download the template for this pull request:
Note
This is auto generated from
Add Template to Pull Request