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

fix/ocp_per_session #495

Merged
merged 13 commits into from
Jun 7, 2024
Merged

fix/ocp_per_session #495

merged 13 commits into from
Jun 7, 2024

Conversation

JarbasAl
Copy link
Member

@JarbasAl JarbasAl commented Jun 4, 2024

tracks available stream extractors per session, this allows running OCP satellite side

NOTE: with hivemind we need to whitelist "ovos.common_play.SEI.get.response" in the listener

2024-06-05 01:21:25.379 - skills - ovos_core.intent_services.ocp_service:handle_get_SEIs:664 - INFO - Session: 5937ce7a-71e3-4d21-8403-142102c5f759 Available stream extractor plugins: ['rss', 'news', 'm3u', 'pls', 'file', 'youtube', 'ydl', 'youtube.channel.live', 'pytube', 'invidious', 'bandcamp']
2024-06-05 01:21:25.384 - skills - ovos_core.intent_services.ocp_service:_search:1301 - DEBUG - Got 3 usable results
2024-06-05 01:21:25.385 - skills - ovos_core.intent_services.ocp_service:handle_play_intent:984 - DEBUG - Playing 3 results for: ozzy osbourne
2024-06-05 01:21:25.385 - skills - ovos_core.intent_services.ocp_service:select_best:1372 - INFO - OVOSCommonPlay selected: skill-ovos-youtube-music.openvoiceos - 94.61538461538461

tracks available stream extractors per session, this allows running OCP in the HiveMind satellite side
tracks available stream extractors per session, this allows running OCP in the HiveMind satellite side
@JarbasAl JarbasAl added the bug Something isn't working label Jun 4, 2024
@JarbasAl JarbasAl requested review from goldyfruit, mikejgray and a team June 4, 2024 23:33
JarbasAl added a commit to JarbasHiveMind/HiveMind-core that referenced this pull request Jun 5, 2024
companion to OpenVoiceOS/ovos-core#495

makes voice sats able to report what stream extractor plugins they have installed for usage with OCP without needing to explicit whitelist the bus message
JarbasAl added a commit to JarbasHiveMind/HiveMind-core that referenced this pull request Jun 5, 2024
companion to OpenVoiceOS/ovos-core#495

makes voice sats able to report what stream extractor plugins they have installed for usage with OCP without needing to explicit whitelist the bus message
to better track and encapsulate session info in a more scalable way
@JarbasAl JarbasAl marked this pull request as draft June 5, 2024 18:30
@JarbasAl JarbasAl changed the title fix/sei_per_session fix/ocp_per_session Jun 5, 2024
@JarbasAl
Copy link
Member Author

JarbasAl commented Jun 5, 2024

this quickly identified some extra issues around session support, marked as draft until i solve those

the player/media state also needs to take session into account

@JarbasAl JarbasAl marked this pull request as ready for review June 6, 2024 05:15
Copy link

codecov bot commented Jun 6, 2024

Codecov Report

Attention: Patch coverage is 74.16667% with 31 lines in your changes missing coverage. Please review.

Project coverage is 73.36%. Comparing base (481c57d) to head (20c4350).
Report is 1 commits behind head on dev.

Files Patch % Lines
ovos_core/intent_services/ocp_service.py 74.16% 31 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #495      +/-   ##
==========================================
+ Coverage   73.18%   73.36%   +0.17%     
==========================================
  Files          15       15              
  Lines        3051     3101      +50     
==========================================
+ Hits         2233     2275      +42     
- Misses        818      826       +8     
Flag Coverage Δ
unittests 73.36% <74.16%> (+0.17%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JarbasAl JarbasAl merged commit e67a798 into dev Jun 7, 2024
3 checks passed
@JarbasAl JarbasAl deleted the fix/sei_per_session branch June 7, 2024 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant