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.

Sign up for GitHub

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

Jump to bottom

Cleanup for 2022 #1

Open

scarmain wants to merge 1 commit into vampjaz:master from scarmain:master

scarmain commented Feb 25, 2022

As the commit says:
fix pot, no display, and forced 10ms loop hit; cleaned up warnings for 2022 compatibility

Our team is using the board, and needed these fixes to work on our robot.

The pot was referring to the Rio port 3, not Mxp port 3 (aka Rio port 7).
We had to send the blink byte to have the display show
We took out the Timer.wait(0.01) in display, as we don't want to sleep the main thread during runtime (effectively the robot loop time of 20ms is our Thread.wait)
Cleaned up a couple warnings and old code formatting


          fix pot, no display, and forced 10ms loop hit; cleaned up warnings fo…

aa84e28

…r 2022 compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet