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 GUI for speech recognition testing and improve recognition accuracy #12

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

petschekr
Copy link

@petschekr petschekr commented Jun 1, 2017

  • Adds a GUI with support for:
    • Recognizing speech from a live recording
    • Recognizing speech from a .wav file
    • Recognizing speech from a folder of .wav files
    • Exporting speech recognition results to a .json file for comparison
  • Improves speech recognition accuracy by applying the recognition threshold relatively to the other matches that the engine returns. Previously, recognition results were only accepted if their probability was over 100% (minimum probability (returned by the engine as the log10 of the actual probability) was set to 0 and 10^0 = 1.00).
  • Fixes inconsistent whitespace and indentation styles in the hlpr_speech_recognition module

@petschekr petschekr changed the title Add GUI for speech recognition testing and improve Add GUI for speech recognition testing and improve recognition accuracy Jun 1, 2017
@petschekr
Copy link
Author

@petschekr
Copy link
Author

@vchu Did you ever get to review this?

@petschekr petschekr requested a review from vchu August 4, 2017 18:45
@vchu
Copy link
Contributor

vchu commented Aug 4, 2017

No I didn't! Missed my radar. Thanks for pointing it out. Well definitely review this. I'll be testing speech this weekend.

@eshort
Copy link
Contributor

eshort commented Aug 6, 2017

I believe this PR is missing the travis fixes that I merged in a few days ago -- can you try pulling in the changes from master and see if the travis build works?

@petschekr
Copy link
Author

Merged in those fixes and the build now passes

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.

3 participants