search

ros-teleop / twist_mux

Twist multiplexer
Apache License 2.0
57 stars 86 forks source link

Foxy updates #25

Closed v-lopez closed 3 years ago

v-lopez commented 3 years ago

Added some arguments and updates for foxy on top of https://github.com/ros-teleop/twist_mux/pull/18

We may want a foxy branch instead of merging to dashing.

bmagyar commented 3 years ago

I've retargeted to foxy-devel. Are these commits accurate? Are you sure you wanted to include all?

v-lopez commented 3 years ago

I branched off #18 and added my thing, I did some super basic test and it works. (ie publishing on one topic and it being republished to other topic).

v-lopez commented 3 years ago

I disabled the test that was failing (test needs migration still) but the code works fine, tested multiple topics with different priorities and joystick priority. Also fixed a segfault when closing and some valgrind complains. Updated format and licenses. Should be good to go.

bmagyar commented 3 years ago

Thank you!