Skip to content

Dev/serial com jbl#288

Open
JoeyBilodeauL wants to merge 16 commits into
release/circ2026from
dev/serial_com_JBL
Open

Dev/serial com jbl#288
JoeyBilodeauL wants to merge 16 commits into
release/circ2026from
dev/serial_com_JBL

Conversation

@JoeyBilodeauL
Copy link
Copy Markdown

class for serial com

Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.cpp Outdated
Comment thread rover_shared_libs/rover_lib2/src/rover_lib2/communication/Serial/serial_com.hpp Outdated
Copy link
Copy Markdown
Contributor

@emilesavoie emilesavoie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are quite a few comments here so I will let you start on this. The basic functionnality is what we want but the structure lacks a bit of finesse

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants