space-concordia-robotics / robotics-prototype

Teleoperated Mars rover with autonomous capabilities intended for robotics competitions.

https://spaceconcordia.github.io/robotics.html

25 stars 18 forks source link

Flow Control #535

Closed VimFreeman closed 3 years ago

VimFreeman commented 3 years ago

Assignee Section

Description

Implemented flow control. Will need to resolve conflicts bc of 42069

Reviewer Section

Aside from local testing and the General Integration Test it is implied that static analysis should be included in the verification process.

[ ] Local Test Performed Successfully
[ ] General Integration Test Performed Successfully

For Pull Requests that do not include code changes, it is not required to perform the tests above.

MartensCedric commented 3 years ago

didn't test, this might not work at all, we might want to test the current comms-master-0 first, or maybe not, your judgement, we're going fast :sunglasses:

VimFreeman commented 3 years ago

didn't test, this might not work at all, we might want to test the current comms-master-0 first, or maybe not, your judgement, we're going fast :sunglasses:

Yes totally should wait before merging, I realize it's missing some stuff that elec had to be done for