velocity_control: allow non-persistent initial velocity commands #2265
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refactor to set model linear and angular velocity commands directly in the OnCmdVel callback. Remove the targetVel member variable and UpdateVelocity method. Signed-off-by: Steve Peters <[email protected]>
Currently the velocity_control system commands velocities in a persistent manner. This change adds a @Persistent attribute (default true) to the //initial_linear and //initial_angular XML tags of the velocity_control system. By setting @Persistent=false, the initial_* values will correspond to initial velocities that may change over time. Any twist values published to the cmd_vel topic will still be interpreted as persistent velocity commands. Signed-off-by: Steve Peters <[email protected]>
Signed-off-by: Steve Peters <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## scpeters/physics_remove_velocity_cmds #2265 +/- ##
========================================================================
Coverage ? 65.84%
========================================================================
Files ? 323
Lines ? 30682
Branches ? 0
========================================================================
Hits ? 20204
Misses ? 10478
Partials ? 0 ☔ View full report in Codecov by Sentry. |
|
I will use a different approach to set initial velocity for a model |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎉 New feature
Specify non-persistent initial velocity
Summary
Currently the
velocity_controlsystem allows specifying initial velocities, but it treats them as persistent velocity commands. This adds a@persistentattribute to the//initial_angularand//initial_linearplugin elements, which can be set to false in order to specify an initial velocity rather than a persistently updated velocity.This pull request builds upon #2228 and targets that branch. This is in draft mode until the example is well-documented and a test is added.
Test it
Run with the
examples/worlds/initial_velocity.sdfexample world and observe different behaviors with different values of the@persistentattribute.Checklist
codecheckpassed (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-bymessages.