-
Notifications
You must be signed in to change notification settings - Fork 147
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 SpawnEntity, DeleteEntity, & SetEntityPose Support #380
base: ros2
Are you sure you want to change the base?
Conversation
@retinfai Did you mean to close this PR? |
I wasn't sure whether I had to have all the things checked off before opening the PR? Or can it be open, while I complete all the tasks |
You can mark it as a draft PR if you're still working on it. Reviewers might give you initial feedback. |
Perfect, thanks |
Is there any updates on this? |
Not that I know if. It would be great to get this PR moving again. |
Functionally everything in this PR works. I've been using it in my projects fine. Just haven't had time to go back and follow the process properly. Will try to move things along |
@retinfai
It prints this, but no response after this,
|
Hmm, not quite sure. Everything looks fine; haven't tried services on things other than a model, so i guess it isn't all working haha. Can you verify that you get the correct expected behaviour calling the service from the gazebo side? |
@retinfai There was some problem in my package. The model is moving fine using the service. Edit |
Co-authored-by: Alejandro Hernández Cordero <[email protected]> Signed-off-by: Afereti Pama <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]> Signed-off-by: Afereti Pama <[email protected]>
Hi, is there any ETA as to when this will be merged? |
@retinfai |
@k12onoss Any solutions for this? I am trying to do the same but get the following response and the service is not displayed when calling |
@elmeripk apologies for late a reply. Yes, I eventually got it working. |
I tested the create/remove/set_pose services on Jazzy and Harmonic and the changes in this PR worked fine for me. |
@retinfai getting this issues.Please can you help? |
When will this get merged? Still the same error as
|
Also looking for this feature.. |
For everyone's awareness, it is already possible to spawn models from ROS 2 using the For DeleteEntity, #629 will be a viable alternative once it's merged. |
That's what I am currently using, but the service would be cleaner imo |
🎉 New feature
Closes #363
Summary
Add support for SpawnEntity, DeleteEntity, & SetEntityPose services
Add support for EntityFactory
Test it
Spin up a parameter bridge for the service you are testing, and make a ros2 service call to it
Example for
SpawnEntity
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.