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

mimic joint plugin added #739

Closed
wants to merge 2 commits into from
Closed

mimic joint plugin added #739

wants to merge 2 commits into from

Conversation

addy1997
Copy link
Contributor

@addy1997 addy1997 commented Apr 2, 2021

Summary

A simple plugin to mimic joint.

Files include

  • MimicJoint.hh
  • MimicJoint.cc
  • CmakeLists.txt
  • README.md (for instructions to generate .so file)

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

…Features (#690)

* GetContactsFromLastStepFeature made optional

Signed-off-by: Tomas Lorente <jtlorente@ekumenlabs.com>

Co-authored-by: Addisu Z. Taddese <addisu@openrobotics.org>
Signed-off-by: addy1997 <adwaitnaik2@gmail.com>
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Apr 2, 2021
@osrf-triage osrf-triage added this to Inbox in Core development Apr 2, 2021
Signed-off-by: addy1997 <adwaitnaik2@gmail.com>
@scpeters
Copy link
Member

scpeters commented Apr 2, 2021

this looks like a port of the following gazebo classic plugin:

there are some API's that are not supported in ignition-gazebo

@addy1997
Copy link
Contributor Author

addy1997 commented Apr 3, 2021

this looks like a port of the following gazebo classic plugin:

* https://github.com/roboticsgroup/roboticsgroup_gazebo_plugins/blob/master/src/mimic_joint_plugin.cpp

there are some API's that are not supported in ignition-gazebo

This is a system plugin. I went through the code twice to examine the namespace/structure/syntax used and it was similar to other (supported) plugins. Could you elaborate more on what exactly in my code or the API needs to be changed in order to make it workable in ign gazebo?

@ahcorde
Copy link
Contributor

ahcorde commented Apr 5, 2021

Hi @addy1997,

I can see that this package is using ROS. This repository should not contain any kind of ROS dependency. This is the strong reason why we should refuse this contribution. This package should live in its own repository.

@addy1997
Copy link
Contributor Author

addy1997 commented Apr 5, 2021

Hi @addy1997,

I can see that this package is using ROS. This repository should not contain any kind of ROS dependency. This is the strong reason why we should refuse this contribution. This package should live in its own repository.

Hi @ahcorde,

Yes, this is a valid point. Is there a repo for ros_ign gazebo plugins where I can contribute this? Actually, this plugin is useful for many applications.

@ahcorde
Copy link
Contributor

ahcorde commented Apr 5, 2021

Here is the ros_ign repository. There are different ROS versions.

I will close this issue.

@ahcorde ahcorde closed this Apr 5, 2021
Core development automation moved this from Inbox to Done Apr 5, 2021
@j-rivero j-rivero removed this from Done in Core development May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants