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

PersonalAudioClassifier extension does not classify #39

Open
1 of 6 tasks
kdclang opened this issue Jun 15, 2020 · 1 comment
Open
1 of 6 tasks

PersonalAudioClassifier extension does not classify #39

kdclang opened this issue Jun 15, 2020 · 1 comment
Labels

Comments

@kdclang
Copy link

kdclang commented Jun 15, 2020

Describe the bug

The PAC extension originally worked with the template https://drive.google.com/open?id=19quTQ9UWVSQ3VfKKh9R-zBiE35wkKy93

Now, although you get a ClassifierReady event, it never triggers a GotClassification event.

Affects

  • Designer
  • Blocks editor
  • Companion
  • Compiled apps
  • Buildserver
  • Other... (please describe)

Expected behavior
With attached aia compiled and installed on device, you should see "Ready to authenticate" label at bottom of screen. When you click on record and record yourself saying "hello" or "goodbye" , the status label should display response from the classifier, and you should hear "Sorry you cannot open my diary".

voice_authentication_diary_complete.aia.zip

Steps to reproduce

Compile attached aia into apk (Companion manifest does not allow use of microphone with aia), install on device, when app is opened, you get "Ready to authenticate" label that signals classifier is ready. Then press Record button and record "hello" or "goodbye". You should see the status label at the bottom change to the response from classifier, but ... nothing happens.

@kdclang kdclang added the bug label Jun 15, 2020
@ewpatton
Copy link
Member

ewpatton commented Sep 4, 2020

This should work with the version I just published on the extensions page. Please update the extension and confirm. It has an updated error block as well that includes an errorMessage, so you'll need to swap in the new error handler in the blocks code.

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

No branches or pull requests

2 participants