Skip to content

Teleop: make default velocities configurable - #778

Open
ntfshard wants to merge 1 commit into
gazebosim:gz-gui8from
ntfshard:gz-gui8-teleop-configurable-defaults
Open

Teleop: make default velocities configurable#778
ntfshard wants to merge 1 commit into
gazebosim:gz-gui8from
ntfshard:gz-gui8-teleop-configurable-defaults

Conversation

@ntfshard

Copy link
Copy Markdown
Contributor

🎉 New feature

Closes #

Summary

Teleop plugin can be used with various types of machines.
And having configurable default velocity values is a quality of life improvement.

image

To spawn plugin, add to world (maybe you also need rest default GUI plugins)

    <gui>
      <plugin filename="Teleop" name="Teleop">
        <gz-gui>
          <property type="string" key="state">docked</property>
        </gz-gui>
        <topic>/cmd_vel</topic>
        <max_forward_velocity>14</max_forward_velocity>
        <max_vertical_velocity>2</max_vertical_velocity>
        <max_yaw_velocity>3</max_yaw_velocity>
      </plugin>
    </gui>

Backport Policy

  • This is safe to backport to the following versions:
    • Jetty
    • Ionic
    • Harmonic
    • Fortress
  • This should not be backported
  • I am not sure
  • Other (fill in yourself)

Test it

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the feature
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Assisted-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

@ntfshard
ntfshard requested a review from jennuine as a code owner August 16, 2026 16:38
@github-actions github-actions Bot added the 🎵 harmonic Gazebo Harmonic label Aug 16, 2026
Signed-off-by: Maksim Derbasov <ntfs.hard@gmail.com>
@ntfshard
ntfshard force-pushed the gz-gui8-teleop-configurable-defaults branch from 7395aee to c89bbb7 Compare August 16, 2026 16:51

@ahcorde ahcorde left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mind to target main?

@ntfshard

Copy link
Copy Markdown
Contributor Author

Do you mind to target main?

Just follow the rules: https://gazebosim.org/docs/all/contributing/#contributing-code

Right now I'm on vacation, and mine development setup is a miles away. I guess I can create a PR to main when I return

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎵 harmonic Gazebo Harmonic

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

3 participants