-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace rclpyHandle with type stubs #1326
Replace rclpyHandle with type stubs #1326
Conversation
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Pulls: #1326 |
Pulls: #1326 |
This one is going to have to be rebased onto the latest to pass CI. Sorry about that. |
Pulls: #1326 |
Currently using
rclpyHandle
to mock the pybind implementation has cause some weirdness when dealing with type annotations. A better and more scalable solutions is replacing it with custom type stubs. One day in the future I would love to use stubgen to automatically generate stubs however it is still missing many features for example generics, TypeDicts, etc. Pybind has made progress in the recent years with function type annotations it however still needs work. This would also allow us to move the VariousHandle
Protocols into type stubs which is probably a better long term plan.