-
Notifications
You must be signed in to change notification settings - Fork 18
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
refactor/padatious_samples #309
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #309 +/- ##
===========================================
- Coverage 50.35% 12.45% -37.90%
===========================================
Files 119 10 -109
Lines 10077 1397 -8680
===========================================
- Hits 5074 174 -4900
+ Misses 5003 1223 -3780 |
Even with this PR, I'm still not able to have the personal skill working. From core logs:
From the personal skill log:
|
Some intents as well are not working, for example: "what is the temperature" works but "Do I need an umbrella" does not. |
also needs OpenVoiceOS/OVOS-workshop#66 |
Does it needs to run with core or with the skill? |
ovos_workshop needs to be updated in the skill, this will emit the new optional data with all samples on the bus instead of filepath ovos-core needs to also be updated to actually consume those samples |
When I install the PR 66 into the skill container I got this.
|
fixed now OpenVoiceOS/OVOS-workshop#66 |
companion to OpenVoiceOS/ovos-core#309
companion to OpenVoiceOS/ovos-core#309
6675ffc
to
3349034
Compare
padatious accepted a file path in the bus message registering intents, this causes issues when using skills standalone via ovos-workshop SkillContainer, the file is not assured to exist in ovos-core
this adds support for messages to send the samples directly, a companion PR in ovos workshop will update the emitted message
backwards compat is maintained with no version sync needed