ros2 / rclpy

rclpy (ROS Client Library for Python)
Apache License 2.0
291 stars 224 forks source link

Add types to wait_for_message.py and moves Handles into type stubs #1325

Closed InvincibleRMC closed 1 month ago

InvincibleRMC commented 2 months ago

This pull request adds types to wait_for_message.py. It also creates the _rclpy.WaitSet. Finally it moves the various Handle objects into _rclpy_pybind11.pyi.

sloretz commented 1 month ago

Pulls: ros2/rclpy#1325 Gist: https://gist.githubusercontent.com/sloretz/e11b408ea5abaf83d4be1cab5bdf4568/raw/e737363e25007355832cccf984cd87285e962c19/ros2.repos BUILD args: --packages-above-and-dependencies rclpy TEST args: --packages-above rclpy ROS Distro: rolling Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14436

sloretz commented 1 month ago

@mergifyio update

mergify[bot] commented 1 month ago

update

✅ Branch has been successfully updated

sloretz commented 1 month ago

Pulls: ros2/rclpy#1325 Gist: https://gist.githubusercontent.com/sloretz/62f40dc912a8c6d90383b8b429f90d84/raw/e737363e25007355832cccf984cd87285e962c19/ros2.repos BUILD args: --packages-above-and-dependencies rclpy TEST args: --packages-above rclpy ROS Distro: rolling Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14437

InvincibleRMC commented 1 month ago

There seems to be a merge problem I will fix when I get a chance.

sloretz commented 1 month ago

@mergifyio update

mergify[bot] commented 1 month ago

update

☑️ Nothing to do

- [ ] `#commits-behind>0` [📌 update requirement] - [X] `-closed` [📌 update requirement] - [X] `-conflict` [📌 update requirement] - [X] `queue-position=-1` [📌 update requirement]
sloretz commented 1 month ago

Pulls: ros2/rclpy#1325 Gist: https://gist.githubusercontent.com/sloretz/f76824db58100e24ade28109e5780bfb/raw/e737363e25007355832cccf984cd87285e962c19/ros2.repos BUILD args: --packages-above-and-dependencies rclpy TEST args: --packages-above rclpy ROS Distro: rolling Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14456