Skip to content

Update real-robots.yml#558

Open
carebare47 wants to merge 3 commits intomasterfrom
B_vive_false_real_robots
Open

Update real-robots.yml#558
carebare47 wants to merge 3 commits intomasterfrom
B_vive_false_real_robots

Conversation

@carebare47
Copy link
Contributor

Proposed changes

Plumb the real_vive=(true|false) oneliner argument into the icon scripts vive:=(true|false) argument passed into sr_teleop_vive_polhemus launch files

Checklist

Before posting a PR ensure that from each of the below categories AT LEAST ONE BOX HAS BEEN CHECKED. If more than one category is applicable then more can be checked. Also ensure that the proposed changes have been filled out with relevant information for reviewers.

Tests

  • No tests required to be added. (For small changes that will be tested by CI/CD infrastructure).
  • Added/Modified automated and PhantomHand CI tests (if a new class is added (Python or C++), the interface of that class must be unit tested).
  • Manually tested in simulation (if simulation specific or no hardware required to test the functionality).
  • Manually tested on hardware (if hardware specific or related).

Documentation

  • No documentation required to be added.
  • Added documentation (For any new feature, explain what it does and how to use it. Write the documentation in a relevant space, e.g. Github, Confluence, etc).
  • Updated documentation (For changes to pre-existing features mentioned in the documentation).

@carebare47 carebare47 requested review from a team as code owners September 13, 2023 14:49
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 this pull request may close these issues.

2 participants