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

Use GUI media service instead of handling gui #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AIIX
Copy link

@AIIX AIIX commented Jan 14, 2022

Description

Type of PR

  • Bugfix
  • Feature implementation
  • Refactor of code (without functional changes)
  • Documentation improvements
  • Test improvements

Testing

  1. Installing the GUI Media Service Audio Backend:
cd mycroft-core
source .venv/bin/activate
pip install git+https://github.com/MycroftAI/plugin-playback-gui-player
  1. changes required add them to /etc/mycroft/mycroft.conf to activate guiplayer audio backend
{
    "Audio": {
        "backends": {
            "local": {
            "type": "guiplayer",
            "active": true
            },
            "vlc": {
            "type": "vlc",
            "active": false,
            "duck": true
            }
        },
        "default-backend": "local"
    }
}
  1. You will also need this PR in skill-playback-control (skill is called mycroft-playback-control.mycroftai on device): Marquee (scrolling) text added and some more UI and metadata fixes for Title and Author, and additional playback fixes on end of media: Add marquee text type and fix metadata for UI skill-playback-control#53
cd /opt/mycroft/mycroft-playback-control.mycroftai
git pull origin pull/53/head

CLA

  • Yes

@krisgesling
Copy link

Hey, this is not intended to be a long term Skill, it's purely for demonstration purposes with the mark-ii/qa branch of things, which is why it's got a very simple self-managed GUI.

I've pushed the changes from this PR to a new feature/gui-media-service so that we can switch to it when we're ready to look at the centralized media management.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants