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.

Sign up for GitHub

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

Jump to bottom

Joint velocity limits #349

Closed
diegoferigo opened this issue May 13, 2021 · 1 comment · Fixed by #356
Closed

Joint velocity limits #349

diegoferigo opened this issue May 13, 2021 · 1 comment · Fixed by #356
Assignees
@diegoferigo
Labels
complexity::low component::scenario::core component::scenario::gazebo issue::resolution::fixed issue::type::announcement issue::type::enhancement

Comments

@diegoferigo
Copy link
Collaborator

We should take some action with gazebo-forks/dart#19 being merged. I'm opening this issue for the following reasons:

  1. Reminder to introduce the support of handling velocity limits (done in feature/velocity_limit but it builds on another branch that I want to merge first).
  2. Notify downstream users that their simulations could change if their models have velocity limits. In fact, before Cherry-pick upstream PR 1407: Add joint velocity limit constraint support gazebo-forks/dart#19, they were not enforced by the physics engine.

In the past, when joint effort limits were enforced, we encountered some problems. Some of the results we've got were no longer valid. I'd like to avoid repeating a similar situation to happen this time.

cc @traversaro @paolo-viceconte @raffaello-camoriano @CarlottaSartore @FirefoxMetzger @AndrejOrsula
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@diegoferigo diegoferigo

Labels
complexity::low component::scenario::core component::scenario::gazebo issue::resolution::fixed issue::type::announcement issue::type::enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support setting joint velocity limits robotology-legacy/gym-ignition
1 participant
@diegoferigo