feat/legacy_audio_api #456
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #456 +/- ##
==========================================
+ Coverage 66.70% 73.09% +6.38%
==========================================
Files 15 15
Lines 2610 2661 +51
==========================================
+ Hits 1741 1945 +204
+ Misses 869 716 -153
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
NeonDaniel
reviewed
May 9, 2024
Merged
JarbasAl
added a commit
to OpenVoiceOS/ovos-ocp-audio-plugin
that referenced
this pull request
May 10, 2024
allow using Classic OCP with the new pipeline companion to OpenVoiceOS/ovos-core#456
|
from the new docs https://openvoiceos.github.io/ovos-technical-manual/OCP_pipeline/
|
JarbasAl
added a commit
to OpenVoiceOS/ovos-ocp-audio-plugin
that referenced
this pull request
May 10, 2024
allow using Classic OCP with the new pipeline companion to OpenVoiceOS/ovos-core#456
JarbasAl
added a commit
to OpenVoiceOS/ovos-ocp-audio-plugin
that referenced
this pull request
Sep 3, 2024
* refactor/utils_0.1.0 (#108) * refactor/utils_0.1.0 * opm imports * no create .desktop file * no create .desktop file * requirements * requirements --------- Co-authored-by: JarbasAi <jarbasai@mailfence.com> * Increment Version to 0.0.7a1 * Update Changelog * refactor/dry_ocp_imports (#109) * refactor/dry_ocp_imports import Playlist and MediaEntry objects from utils 0.1.0 * tests * tests * fix/ocp_playlist allow initing Playlist object as a regular list --------- Co-authored-by: JarbasAi <jarbasai@mailfence.com> * Increment Version to 0.0.7a2 * Update Changelog * feat/experimental_ocp_pipeline (#112) allow using Classic OCP with the new pipeline companion to OpenVoiceOS/ovos-core#456 * Increment Version to 0.0.7a3 * Update Changelog * fix/coexistence_with_pipeline (#115) * fix/coexistence_with_pipeline intents were being registered even if pipeline enabled, only the media matchers were taking into account the new flag this commit moves the check to init instead * port/SEI report event report plugins to ovos-core like ovos-media does to help in transition * fix stop * Increment Version to 0.0.7a4 * Update Changelog * fix/playlist handling (#116) * fix deprecation warnings * fix type checks * better log * support older ovos-utils * type checking fixes * typing * yeah * Increment Version to 0.0.7a5 * Update Changelog * fix/playlist_again (#117) * Increment Version to 0.0.7a6 * Update Changelog * default to OCP pipeline + support for utils 0.0.38 (#119) * remove dead code since OpenVoiceOS/ovos-core#491 and OpenVoiceOS/ovos-config#96 this is effectively dead code all NLP matching is dropped in this PR * rm dead code * drop old tests * Increment Version to 0.0.7a7 * Update Changelog * fix/native_sources (#120) * fix/native_sources allow usage with voice satellites, without triggering playback on master side similar to what classic audio service does, but with a decorator to make it more usable * handle message=None * fix backend check * kwarg * add helper property * message None * default val * missing kwarg * drop old classifier tests * drop old classifier tests * keep your crappy test * keep your crappy test * a relevant test now * Increment Version to 0.0.7a8 * Update Changelog * fix/play_from_gui (#121) * fix/play_from_gui when selecting entries from featured media PluginStream was not being recognized if audio restarts with GUI opening, playback would fail due to GUI sending malformed data, this commit also works around that issue * more fixes * more fixes * update tests * restore tst * keep old OCP bugs around 👍 * fixy fixes * Increment Version to 0.0.7a9 * Update Changelog * fix/playlist_handling (#122) * fix/playlist_handling fix position of "now playing" handling fix PluginStream to MediaEntry extraction fix Playlist of PluginStream * deprecate extract_stream * check bounds of idx * remove test for old ovos-audio version compat with utils 0.0.38 * fix tests * Increment Version to 0.0.7a10 * Update Changelog * Update README.md * logs/end_of_track (#124) make it easier to debug issues like #123 * Increment Version to 0.0.7a11 * Update Changelog * Increment Version to 0.0.7 * Update Changelog --------- Co-authored-by: NeonJarbas <59943014+NeonJarbas@users.noreply.github.com> Co-authored-by: JarbasAi <jarbasai@mailfence.com> Co-authored-by: JarbasAl <JarbasAl@users.noreply.github.com> Co-authored-by: JarbasAI <33701864+JarbasAl@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #455
makes OCP skills usable without the OCP plugin
test with
more info on the legacy system here: https://github.com/OpenVoiceOS/ovos-audio?tab=readme-ov-file#using-legacy-audioservice