-
Notifications
You must be signed in to change notification settings - Fork 17
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
fix/speech #47
fix/speech #47
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Goodbye, regex. I don't really get the removed hack. Was that compensating for Picroft, Pi, or Raspbian, and do we care?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, will follow all related PR's, bump, test and report back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If removed patch is no longer patching an RPi issue, this seems fine; otherwise consider putting it in a method with a config option to suppress splitting sentences
Testing this PR however following error is for all TTS output.
Used the default mycroft setup route utilizing the Mimic2 and selene pairing. |
Following error is within voice.log but not sure if this related.
|
thats mimic2 plugin failing, all the errors are just logs but actually handled in code, try out OpenVoiceOS/ovos-plugin-manager#27 and see if it improves things, if fallback TTS is not triggering with that + #49 then we have a new issue the second thing is a new issue, OpenVoiceOS/ovos-listener#22 |
bumps ovos utils to include OpenVoiceOS/ovos-utils#14
bumps ovos plugin manager to include OpenVoiceOS/ovos-plugin-manager#26
removes a hack with a TODO
refactors out the global config variable
wraps the fallback tts into a try: except: