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.

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

Update show tablet package name #88

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

hello-lamsey
Copy link
Contributor

@hello-lamsey hello-lamsey commented Jul 29, 2024

Description

Changes references to the stretch_show_tablet packages to reflect new naming convention.

Testing procedure

Rebuild ROS2 workspace, run web teleop, and try to show someone the tablet.

Before opening a pull request

From the top-level of this repository, run:

  • pre-commit run --all-files

To merge

  • Squash & Merge

@hello-lamsey hello-lamsey changed the title pdate show tablet package name Update show tablet package name Jul 29, 2024
@hello-vinitha hello-vinitha self-assigned this Jul 31, 2024
@hello-vinitha
Copy link
Collaborator

Verified this works on 3030.

@hello-vinitha hello-vinitha merged commit 6963910 into vinitha/tablet_placement Jul 31, 2024
1 check passed
@hello-vinitha hello-vinitha deleted the matt/rename_show_tablet branch July 31, 2024 20:40
hello-amal added a commit that referenced this pull request Jul 31, 2024
* [WIP] integrate show tablet into web app

* Got toggle body pose overlay checkbox working

* Web app UX is working, with dummy show tablet action

* Finalize integration

* Ignore stale body poses

* Reset to navigation mode after invoking autonomous clients

* Remove dummy action server

* update show tablet package name (#88)

---------

Co-authored-by: Matt Lamsey <[email protected]>
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