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

better error logs #54

Merged
merged 1 commit into from
Feb 11, 2024
Merged

better error logs #54

merged 1 commit into from
Feb 11, 2024

Conversation

JarbasAl
Copy link
Member

when no fallback tts is loaded dont try to execute TTS, log an error and return instead

makes logs more readable, instead of the error below hiding the real issue

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/pi/.venvs/ovos/lib/python3.9/site-packages/ovos_audio/service.py", line 380, in execute_fallback_tts
    tts.execute(utterance, ident, listen,
AttributeError: 'NoneType' object has no attribute 'execute'

when no fallback tts is loaded dont try to execute TTS, log an error and return instead

makes logs more readable, instead of the error below hiding the real issue

```
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/pi/.venvs/ovos/lib/python3.9/site-packages/ovos_audio/service.py", line 380, in execute_fallback_tts
    tts.execute(utterance, ident, listen,
AttributeError: 'NoneType' object has no attribute 'execute'
```
@JarbasAl JarbasAl added the enhancement New feature or request label Feb 11, 2024
@JarbasAl JarbasAl requested review from mikejgray and a team February 11, 2024 18:00
@JarbasAl JarbasAl merged commit 983256d into dev Feb 11, 2024
4 checks passed
@JarbasAl JarbasAl deleted the better-error-logs branch February 11, 2024 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants