ROS Resources: Documentation | Support | Discussion Forum | Index | Service Status | ros @ Robotics Stack Exchange
Ask Your Question
1

[ROS2] rclpy Executor spin_some alternative/support

asked 2019-12-17 02:53:47 -0600

DreamEquation gravatar image

Not sure if this should be posted here or on the github issue, but I would like to know if there are any ways to achieve the spin_some behaviour of rclcpp in rclpy? If not, then I would gladly attempt to submit a PR to implement the behaviour if needed.

edit retag flag offensive close merge delete

Comments

Currently some work is planned/being discussed to refactor how the executor works in ROS2 for the Foxy release. Discussion can be found here: https://discourse.ros.org/t/singlethr... . The focus will be on rclcpp, but if changes are made in rcl this could also require some rework of rclpy (I'm not very familiar with the rclpy implementation, but one can assume). So a bit of a heads-up w.r.t. possible API changes coming up (you may have to rebase with master a couple of times for the duration of the PR).

If you are really interested in working with the executor (be it in rclcpp or rclpy) you could voice this on discourse (pretty sure the core ROS2 guys will appreciate the offer/help).

If you just want the spin_some functionality in rclpy, then creating a PR for it is of course also fine ^^ , just thought ...(more)

MCornelis gravatar image MCornelis  ( 2019-12-18 04:27:09 -0600 )edit

thanks for the heads-up!

DreamEquation gravatar image DreamEquation  ( 2019-12-18 04:43:47 -0600 )edit

1 Answer

Sort by ยป oldest newest most voted
1

answered 2019-12-17 15:28:03 -0600

johnconn gravatar image

It doesn't look like it's implemented in rclpy. I recommend filing an issue on github and then going after that PR you mention.

rclcpp spin_some recently changed to only check for work once, so make sure to replicate that in your PR.

edit flag offensive delete link more

Question Tools

2 followers

Stats

Asked: 2019-12-17 02:53:47 -0600

Seen: 916 times

Last updated: Dec 17 '19