[N/A] Expose height in Spot_Wrapper velocity command

[fgillespie-bdai]

Change Overview

This change enables dynamic height changes for teleop during the Drive-A-Spot demo. It adds an optional body_height parameter for the velocity_cmd function in Spot Wrapper.

Testing Done

This change has been tested on Spot.

Instructions for Reviewers

• Certify that this change is following our dependency rules.
• Certify that the author has filled in the testing section with appropriate testing:
  • Spot testing for PRs touching Spot functionality, including the list of functions tested.
  • Open AI testing for PRs touching code that calls Open AI functions, including a list of tests done. For example, did they test dry, canned, AND online?
  • ROS 2 testing for PRs that use ROS 2 and that their ROS 2 tests use the domain_coordinator to prevent port conflicts.
• For changes to docker, take a look at docker testing

[fgillespie-bdai]

• [N/A] Expose height in Spot_Wrapper velocity command #147 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fgillespie-bdai and the rest of your teammates on Graphite

[coveralls]

Pull Request Test Coverage Report for Build 11391430677

Details

• 1 of 4 (25.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.01%) to 40.835%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
spot_wrapper/wrapper.py	1	4	25.0%

Totals
Change from base Build 11386831056:	-0.01%
Covered Lines:	1869
Relevant Lines:	4577

---

💛 - Coveralls

[tcappellari-bdai]

lgtm

[fgillespie-bdai]

Merge activity

• Oct 18, 1:38 PM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 18, 1:41 PM EDT: Graphite couldn't merge this PR because it failed for an unknown reason.
• Oct 18, 1:43 PM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 18, 1:47 PM EDT: Graphite couldn't merge this PR because it failed for an unknown reason.
• Oct 18, 1:48 PM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 18, 1:48 PM EDT: A user merged this pull request with Graphite.