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

Add a script to configure audio devices #77

Merged
merged 4 commits into from
Jul 18, 2024

Conversation

hello-amal
Copy link
Collaborator

@hello-amal hello-amal commented Jul 18, 2024

Description

#62 and #64 added bi-directional audio capabilities to the web interface. However, this working is contingent on proper configuration of (possibly external) audio devices.

This PR adds README instructions for configuring audio devices, and adds a convenience bash script for configuring audio devices.

Testing procedure

  • Have no external audio devices plugged in. Verify that the physical volume knob on the Stretch3's head is turned all the way up. Run ./configure_audio.sh and then launch the interface. Verify both robot-to-operator audio streaming and operator-to-robot TTS work.
  • Plug in an external microphone. Run ./configure_audio.sh and then launch the interface. Verify that robot-to-operator audio streaming uses the new mic, and that operator-to-robot TTS works.
  • Disconnect the above mic and plug in another external mic. Repeat the above test.

Before opening a pull request

From the top-level of this repository, run:

  • pre-commit run --all-files

To merge

  • Squash & Merge

@hello-amal hello-amal requested a review from hello-vinitha July 18, 2024 19:38
@hello-amal hello-amal merged commit 755ddba into master Jul 18, 2024
1 check passed
@hello-amal hello-amal deleted the amaln/default_audio_device branch July 18, 2024 20:30
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.

1 participant