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 trigger sound confirmation #60

Merged
merged 3 commits into from May 22, 2017

Conversation

Projects
None yet
6 participants
@puboe
Contributor

puboe commented May 21, 2017

Add an optional trigger sound confirmation, useful to know when it's actually listening when not using the voice hat hardware or any other led light.

@googlebot googlebot added the cla: yes label May 21, 2017

puboe added some commits May 21, 2017

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io May 21, 2017

Codecov Report

Merging #60 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #60   +/-   ##
======================================
  Coverage    79.2%   79.2%           
======================================
  Files           5       5           
  Lines         202     202           
======================================
  Hits          160     160           
  Misses         42      42

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4169573...5f239de. Read the comment docs.

codecov-io commented May 21, 2017

Codecov Report

Merging #60 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #60   +/-   ##
======================================
  Coverage    79.2%   79.2%           
======================================
  Files           5       5           
  Lines         202     202           
======================================
  Hits          160     160           
  Misses         42      42

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4169573...5f239de. Read the comment docs.

@drigz drigz merged commit 581042e into google:master May 22, 2017

4 checks passed

cla/google All necessary CLAs are signed
codecov/patch Coverage not affected when comparing 4169573...5f239de
Details
codecov/project 79.2% remains the same compared to 4169573
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@drigz

This comment has been minimized.

Show comment
Hide comment
@drigz

drigz May 22, 2017

Member

Thanks very much!

Member

drigz commented May 22, 2017

Thanks very much!

@puboe puboe deleted the puboe:add-trigger-sound branch May 22, 2017

Rexypoo added a commit to Rexypoo/aiyprojects-raspbian that referenced this pull request Jun 1, 2017

Add trigger sound confirmation (#60)
Add an optional trigger sound confirmation, useful to know when it's actually listening when not using the voice hat hardware or any other led light.
@TwoTracks

This comment has been minimized.

Show comment
Hide comment
@TwoTracks

TwoTracks Oct 4, 2017

How would I set this up?

TwoTracks commented Oct 4, 2017

How would I set this up?

@divx118

This comment has been minimized.

Show comment
Hide comment
@divx118

divx118 Oct 5, 2017

Contributor

@TwoTracks on the old API so the master branch, I think the way would be to uncomment trigger-sound = path_to_your_sound.wav in ~./config/voice-recognizer.ini.default and fill in the path to your wav file. On the new API so the voicekit branch it is a bit different see the source here https://github.com/google/aiyprojects-raspbian/blob/voicekit/src/aiy/voicehat.py#L97

Contributor

divx118 commented Oct 5, 2017

@TwoTracks on the old API so the master branch, I think the way would be to uncomment trigger-sound = path_to_your_sound.wav in ~./config/voice-recognizer.ini.default and fill in the path to your wav file. On the new API so the voicekit branch it is a bit different see the source here https://github.com/google/aiyprojects-raspbian/blob/voicekit/src/aiy/voicehat.py#L97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment