Skip to content
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

Add Python bindings #387

Closed
srmainwaring opened this issue Feb 24, 2023 · 2 comments · Fixed by #411
Closed

Add Python bindings #387

srmainwaring opened this issue Feb 24, 2023 · 2 comments · Fixed by #411
Assignees
Labels
enhancement New feature or request

Comments

@srmainwaring
Copy link

Desired behavior

Add Python bindings to gz-transport (and gz-msgs).

Alternatives considered

  • Use ros_gz bridge and the Python support in ROS / ROS2.

Implementation suggestion

Upstream the bindings for gz-msgs and gz-transport from https://github.com/srmainwaring/gz-python.

Additional context

From time to time there are requests for Python bindings to control the simulation. A number of the requirements can be met by using the messaging infrastructure to control and interact with the simulation.

@srmainwaring srmainwaring added the enhancement New feature or request label Feb 24, 2023
@azeey
Copy link
Contributor

azeey commented Feb 24, 2023

Thanks for the implementation suggestion @srmainwaring. This is something we are hoping to address for Harmonic and having your implementation as a starting point will go a long way. We'd also like to address gazebosim/gz-msgs#113, gazebosim/gz-msgs#36, and gazebosim/gz-sim#494 which might have implications for how we implement this.

@azeey
Copy link
Contributor

azeey commented Aug 29, 2023

This has been implemented in #411

@azeey azeey closed this as completed Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants