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

Effort Values are 0 #169

Open
haider8645 opened this issue Aug 7, 2023 · 6 comments · Fixed by #186
Open

Effort Values are 0 #169

haider8645 opened this issue Aug 7, 2023 · 6 comments · Fixed by #186

Comments

@haider8645
Copy link

haider8645 commented Aug 7, 2023

Hello,

the gz_ros2_control is returning 0 values for the joint state efforts. I saw that the gz_ros2_control has commented out these lines in https://github.com/ros-controls/gz_ros2_control/blob/cf5c6a4135d6b46c95e0e135ffd0400fb4db7731/ign_ros2_control/src/ign_system.cpp#L499](ign_system.cpp

Will the effort values be available at some point in time in the future?

Best,
Haider

- interface_names:
  - position
  - velocity
  - effort
  values:
  - 1.3656909652303677e-10
  - 5.423202756300551e-10
  - 0.0
@bmagyar
Copy link
Member

bmagyar commented Aug 7, 2023

Have you tried adding them?

@haider8645
Copy link
Author

What do you mean?

@Luigip98
Copy link

Any update about that? @haider8645

@christophfroehlich
Copy link
Contributor

gazebosim/gz-physics#124 was fixed but not implemented in gz_ros2_control. Happy to review PRs.

@azeey
Copy link
Contributor

azeey commented May 20, 2024

@christophfroehlich #186 has an implementation. Is it not complete?

@christophfroehlich
Copy link
Contributor

@christophfroehlich #186 has an implementation. Is it not complete?

I forgot about that PR tbh. But the author did not respond to the requested changes.. Maybe someone can help out with the hardcoded z-component issue there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants