-
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
feat/pipeline #336
feat/pipeline #336
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #336 +/- ##
==========================================
- Coverage 50.35% 49.41% -0.94%
==========================================
Files 119 12 -107
Lines 10077 1623 -8454
==========================================
- Hits 5074 802 -4272
+ Misses 5003 821 -4182 |
@@ -199,6 +223,38 @@ def disambiguate_lang(message): | |||
|
|||
return default_lang | |||
|
|||
def get_pipeline(self, skip_converse=False, skip_fallback=False): | |||
# Create matchers | |||
# TODO - from plugins |
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.
will solve this TODO in #310
for now just want to make it configurable to clean up padatious/padacioso and allow more control for users
- pipeline settings from OpenVoiceOS/ovos-core#336 - default to padatious again - homescreen enabled by default in GUI - mycroft-gui added to native sources (audio)
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.
A couple of minor nit-picks and things that confused me :)
relevant docs in default config here https://github.com/OpenVoiceOS/ovos-config/pull/63/files#diff-97d1dc6bf78002315ed82919af2d2f7438d7c6fb07f6fff7b8df14026582a93bR123 |
@@ -6,13 +6,13 @@ combo-lock>=0.2.2, <0.3 | |||
padacioso~=0.2, >=0.2.1a8 | |||
adapt-parser>=1.0.0, <2.0.0 | |||
|
|||
ovos_bus_client<0.1.0, >=0.0.5 | |||
ovos_bus_client<0.0.6a1, >=0.0.5 |
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.
TODO OpenVoiceOS/ovos-bus-client@6eb7c56 introduced some issue where unittests hang forever test/unittests/skills/test_event_container.py .... [ 34%]
configurable intent stage