Working code for testing apr 14 (mostly) + running ml works now!#157
Working code for testing apr 14 (mostly) + running ml works now!#157
Conversation
| FREQUENCY = 60 # times per second | ||
| BAUD_RATE = 9600 | ||
| CONNECTION_NAME = "/dev/ttyACM0" | ||
| CONNECTION_NAME = "/dev/ttyACM1" |
There was a problem hiding this comment.
TODO: This should be configurable via the launch file. Kinda janky that we have to change this variable in the code.
imzaynb
left a comment
imzaynb
left a comment
There was a problem hiding this comment.
Looks good to me tho see if you can revert the changes on the file I suggested. Thanks!!
There was a problem hiding this comment.
are you able to revert the changes on this file?
There was a problem hiding this comment.
TODO: look into the get_time_of_day thing that I had to remove from here, kinda sus ngl...
imzaynb
left a comment
imzaynb
left a comment
There was a problem hiding this comment.
It looks like nothing has changed since the last time that I saw this code. I feel kind of sussed out that some of my localization changes are also on this branch, but so long as it has been compiling for you then we can just move forward with the merge.
I'll be merging in my localization branch in the next couple of weeks. Just need to do some finishing touches.
3 participants
No description provided.