Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

Update to the Assistant API v1alpha2 #235

Closed
wants to merge 21 commits into from

Conversation

drigz
Copy link
Member

@drigz drigz commented Jan 4, 2018

Also, fix some style issues from previous changes.

PeterMalkin and others added 18 commits December 18, 2017 13:58
This commit extends the original AIY projects
python API to include drivers, APIs and demos
for the AIY Projects Vision Kit.

https://aiyprojects.withgoogle.com/vision/
This lets us do effects, glissandos, phasing, and so on through a nice
and easy python API. Additionally, it adds the toneplayer and
trackplayer classes as well as the tracks to the setup.py script to make
sure they get deployed.
The v1beta1 endpoint has been shut down, so this is necessary for using
the Cloud Speech demo now.

This is a cherry-pick from the voicekit branch.

See google#205.
Summary of changes:
- Fix instructions for using the Voice Kit on Stretch (google#187)
- Make it easier to turn the LED off (google#167)
- Fix crash with trigger sound (google#163)
- Let user code control TTS (google#185)
- Enable hotwording with Cloud Speech (0b4972b)
Restructures for Vision/Voice kit coexistence.
Restore changes from the voicekit branch
Change-Id: I687ca96e4cf768617fa45d50d68dadffde750b87
Change-Id: I7276ec84ae22adcca3262ad28745302e25bd9a40
Add object_prob_threshold to restrict probabilities, change
max_num_objects default to return all detected objects. Default
behaviour now is to return all detected classes with non-zero
probabilities.

Change-Id: Ibbf97d04dc609bf25c0d5329b7757d1391c9b7cf
Grayscale images are interpreted by VisionBonnet as RGB where all
channels are the same. This is just an optimization to send 3 times less
data over SPI.

Change-Id: I0322cbff8381f295772b1eadead0a4d0d7f335fc
Adds support for Voice V2 and enables the driver for V1.

Change-Id: I5b5c861aca65b6a25e250dbda78535820cbbb969
Change-Id: I33131a033cdf0ffdce9b4ccbe6a67e88aa54cce2
* add immediate option for cloudspeech recognize

* add documentation for immediate argument
@drigz drigz requested a review from proppy January 4, 2018 16:52
The root README.md is documentation for the GitHub repo, shown on the
https://github.com/google/aiyprojects-raspbian. src/README.md is
documentation for the API, which can't contain relative links to files
in the GitHub repo.

Fixes google#214.
@mrbrooksatgoogle
Copy link

This is into the voicekit branch instead of aiyprojects. IIRC, that branch is deprecated.

drigz and others added 2 commits January 5, 2018 11:57
This doesn't update from v1alpha1 of the gRPC API to v1alpha2, which is
to follow.

Change-Id: I86a5f2eb73acf51a169ff678af03f24b4d3e1175
Change-Id: Ic51bee63aeb3117f12b6bacfc5452b055b003b7d
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again. If the bot doesn't comment, it means it doesn't think anything has changed.

@googlebot googlebot added cla: no and removed cla: yes labels Jan 5, 2018
@drigz
Copy link
Member Author

drigz commented Jan 5, 2018

Moved to internal pull request.

@drigz drigz closed this Jan 5, 2018
@drigz drigz deleted the upgrade-assistant branch January 5, 2018 12:20
@drigz drigz restored the upgrade-assistant branch January 5, 2018 12:20
@deviantony
Copy link

deviantony commented Jan 13, 2018

FYI I was able to use the drigz:upgrade-assistant branch from drigz repository with some small tweaks to be abl to speak to the google assistant in French (using the assistant_grpc_demo.py). Great job @drigz ! Any chance to see this merged soon ?

@sheridat
Copy link

Hi, would you please share your "small tweaks"?

@deviantony
Copy link

@sheridat sure, they're available here: https://github.com/deviantony/aiyprojects-raspbian/tree/drigz-upgrade-assistant

Basically, I just changed a few things on top of the changes of @drigz:

Had to run sudo pip install -r requirements.txt to get the latest deps.

Note that this is just some dirty hacking but it seems to work well.

@sheridat
Copy link

@deviantony Thank You very much!

@Jaxom99
Copy link

Jaxom99 commented Mar 13, 2018

Hello @deviantony I'm also trying to get some french speech recognition from the Google Assistant, using the AIY Kit and a RPi3. I cloned your repo, re-run the install scrips and everything, but the examples (including assistant_grpc_demo.py) still speak English.
Is there something I'm missing ? Thanks for your answer.

@deviantony
Copy link

@Jaxom99 have you installed the latest deps via pip ? My fork might be broken using the latest deps and I cannot try it at this time as I do not have access my AIY box.

@Jaxom99
Copy link

Jaxom99 commented Mar 15, 2018

thanks for getting back @deviantony. Yes I used sudo pip install -r requirements.txt as you mentionned. But the i18n.py setting was still useless. Somebody advised to actually call aiy.i18n.set_language_code('fr-FR') in the beggining of the script...
Anyhow I went the easy way, reinstalled the provided image (from february) and used this setting : it speaks and recognize French ! But the "Hacking.md" way of use seems really broken.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet