Skip to content

Conversation

@jujuz455
Copy link
Contributor

Refactored the SRT client app.
It is now a C++ Node with ROS 2 dependencies configured in CMake.
Please let me know if this aligns with the requirements.

Summery:

Ported the client logic to C++.

Wrapped it in a ROS 2 node

Created a launch file (srt_client.launch.py)

Updated CMakeLists.txt

@jujuz455 jujuz455 requested a review from ConnorNeed January 20, 2026 19:28
@ConnorNeed
Copy link
Member

Yep that meets the requirements. We will have to test the pipeline before we merge.

Copy link
Member

@ConnorNeed ConnorNeed left a comment

Choose a reason for hiding this comment

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

Looks good to me

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants