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

Port supported languages API from Neon speech/audio modules #190

Merged
merged 2 commits into from Aug 26, 2022

Conversation

NeonDaniel
Copy link
Member

No description provided.

@NeonDaniel
Copy link
Member Author

@codecov
Copy link

codecov bot commented Aug 26, 2022

Codecov Report

Merging #190 (50c3b03) into dev (6ceb058) will increase coverage by 3.83%.
The diff coverage is 37.52%.

Current head 50c3b03 differs from pull request most recent head 5c6b36a. Consider uploading reports for the commit 5c6b36a to get more accurate results

@@            Coverage Diff             @@
##              dev     #190      +/-   ##
==========================================
+ Coverage   50.35%   54.18%   +3.83%     
==========================================
  Files         119      152      +33     
  Lines       10077     9543     -534     
==========================================
+ Hits         5074     5171      +97     
+ Misses       5003     4372     -631     
Impacted Files Coverage Δ
mycroft/audio/__main__.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/__main__.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/arduino.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/eyes.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/mouth.py 0.00% <0.00%> (ø)
mycroft/client/speech/__main__.py 0.00% <0.00%> (ø)
mycroft/client/speech/hotword_factory.py 0.00% <0.00%> (-88.89%) ⬇️
mycroft/client/speech/service.py 0.00% <0.00%> (ø)
mycroft/client/speech/silence.py 0.00% <0.00%> (-42.86%) ⬇️
mycroft/client/text/__init__.py 0.00% <0.00%> (ø)
... and 138 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

mycroft/audio/service.py Outdated Show resolved Hide resolved
mycroft/audio/service.py Outdated Show resolved Hide resolved
mycroft/listener/service.py Outdated Show resolved Hide resolved
mycroft/listener/service.py Outdated Show resolved Hide resolved
@JarbasAl JarbasAl merged commit 2ffdebe into dev Aug 26, 2022
3 checks passed
@JarbasAl JarbasAl deleted the FEAT_SupportedLanguagesAPI branch August 26, 2022 19:18
@JarbasAl JarbasAl added the enhancement New feature or request label Aug 26, 2022
NeonDaniel pushed a commit to NeonGeckoCom/neon_speech that referenced this pull request Aug 26, 2022
NeonDaniel pushed a commit to NeonDaniel/neon_audio that referenced this pull request Aug 26, 2022
NeonDaniel added a commit to NeonGeckoCom/neon_speech that referenced this pull request Aug 26, 2022
* Add bus API method to get supported languages with unit test

* Fix unit test to account for default return value

* Fix mocking in API tests

* Troubleshooting default value
Add docstring to added method

* Refactor changes on OpenVoiceOS/ovos-core#190

* Update missed message change in unit tests

* Update missed message param change in unit tests

* Cleanup changes

Co-authored-by: Daniel McKnight <daniel@neon.ai>
NeonDaniel added a commit to NeonGeckoCom/neon_audio that referenced this pull request Aug 26, 2022
* Add neon.get_languages_tts event handler with unit test
Update Docker ovos config

* Rebase changes on OpenVoiceOS/ovos-core#190

* Fix ovos-core dependency spec

* Troubleshooting dependency resolution bug in GH actions

Co-authored-by: Daniel McKnight <daniel@neon.ai>
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.

None yet

2 participants