Ask Your Question
0

[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 imageMCornelis ( 2019-12-18 04:27:09 -0600 )edit

thanks for the heads-up!

DreamEquation gravatar imageDreamEquation ( 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

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Question Tools

2 followers

Stats

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

Seen: 38 times

Last updated: Dec 17 '19