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

speaker support #127

Merged
merged 99 commits into from
Mar 6, 2025
Merged

speaker support #127

merged 99 commits into from
Mar 6, 2025

Conversation

karmanyaahm
Copy link
Collaborator

@karmanyaahm karmanyaahm commented Feb 22, 2025

PR Summary

PR Link: INSERT-LINK-HERE

Issue Link: INSERT-LINK-HERE

Description

Add a single line summary describing the purpose of this PR.

Reviewers

Tag reviewers.


Changelog

  • publish tts and music topics when function called in tank_robot_config
  • make the audoi node with chatgpt and set volume stuff

Reviewer Guide

this shouldn't break anything. The speaker code isn't written safely cuz it's a seperate node so even if it breaks we don't care

Testing

Automatic

no

Manual

PXL_20250306_071930447.TS.mp4

Documentation

no

Checklist

  • Confirmed all tests pass on a clean build
  • Added reviewers in Github
  • Posted PR Summary to Discord PR's Channel
  • Ran uncrustify on any modified C++ files
  • Ran Colcon Lint for any modified CMakeLists.txt or Package.xml

Omega Jerry and others added 30 commits January 1, 1970 00:00
fix gains
good move to pose
both tested on hardware
adds intake hook rejection
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
…-GHOST/VEXU_GHOST into hw-test/move-to-pose-tuning-2-18-25
@karmanyaahm karmanyaahm changed the base branch from develop to hw-test/move-to-pose-tuning-2-18-25 February 22, 2025 15:11
@karmanyaahm karmanyaahm changed the base branch from hw-test/move-to-pose-tuning-2-18-25 to develop March 3, 2025 02:23
@karmanyaahm karmanyaahm requested a review from JakeWendling March 6, 2025 07:23
Copy link
Collaborator

@JakeWendling JakeWendling left a comment

Choose a reason for hiding this comment

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

lgtm..................

@karmanyaahm karmanyaahm merged commit 2d5f742 into develop Mar 6, 2025
1 check passed
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.

4 participants