-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Code Enchancement] Support for ovos config #65
Conversation
@NeonDaniel please review it, but overall I think I will have to split up pyklatchat modules, so it will be easier to manage |
@@ -0,0 +1,7 @@ | |||
import importlib | |||
import logging | |||
|
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.
Any reason this isn't just from ovos_utils.log import LOG
? That also handles the log overrides from config so the combo_lock
patching shouldn't be necessary (just addded to default config)
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.
Any reason this isn't just
from ovos_utils.log import LOG
? That also handles the log overrides from config so thecombo_lock
patching shouldn't be necessary (just addded to default config)
I see that in config we only set certain level of those, while I want to disable it completely...
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.
I think errors or at least exceptions should be shown to debug failures
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.
I think errors or at least exceptions should be shown to debug failures
Okay, I think I will leave it as is for now, and focus on implementing the features Elon and Richard are asking me for, and will get back to that later
I see the automation is failing
|
@@ -0,0 +1,7 @@ | |||
import importlib | |||
import logging | |||
|
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.
I think errors or at least exceptions should be shown to debug failures
No description provided.