colla69 / plexmusic-skill Goto Github PK

When I ask for a playlist, the playback fails and debug suggests that it's sending just the song titles. It looks like there should be a loop to discover the uri of each title as though they were individual requests, and then send the array to vlc service.

Also, when this occurs, no further playback requests are sucessful until restarting mycroft skills.

It looks like this is a dead project which is a shame. Hopefully you will revisit it when plex finally releases a headless plexamp player that may allow for http requests.

2021-10-03 19:25:27.451 | DEBUG | 737 | mycroft.messagebus.service.event_handler:on_message:38 | {"type": "play:start", "data": {"skill_id": "plexmusic-skill.colla69", "phrase": "BILLIONS CLUB", "callback_data": {"title": "BILLIONS CLUB", "file": ["I Am...Sasha Fierce [DE] CD1", "Future Nostalgia + Club Future Nostalgia", "Perception", "Luv Is Rage 2", "Beerbongs & Bentleys", "Beerbongs & Bentleys", "Manic", "DAMN.", "Beerbongs & Bentleys", "When We All Fall Asleep, Where Do We Go?", "American Teen", "Stoney", "Please Excuse Me For Being Antisocial", "When We All Fall Asleep, Where Do We Go?", "American Teen", "Pray for the Wicked", "thank u, next", "Scorpion", "Beauty Behind the Madness", "Blurryface", "Evolve", "Sweetener", "AM", "Evolve", "thank u, next", "Expectations", "Scorpion", "Blurryface", "After Hours", "Dua Lipa (deluxe edition)", "Blurryface", "Dua Lipa (deluxe edition)", "Camila", "Night Visions", "Greatest Hits", "A Night At The Opera", "24k Magic", "Night Visions", "Divinely Uninspired To A Hellish Extent", "True", "Greatest Hits", "Beauty Behind the Madness", "Jordi", "Divinely Uninspired To A Hellish Extent", "Fine Line", "Illuminate", "Hozier", "This Is Acting", "Purpose", "A Rush Of Blood To The Head", "Handwritten", "1000 Forms Of Fear", "Purpose", "In The Lonely Hour", "V", "X", "Escape", "Uptown Special", "Love In The Future", "In The Lonely Hour", "Unorthodox Jukebox", "Purpose", "X", "Doo-Wops & Hooligans", "Hot Fuss", "All The Little Lights"]}}, "context": {}}

I like your skill but it is kind of complicated to use if you want use other media Skills because your skill always answers. For example if I say "play the movie guardians of the galaxy" for the cpkodi skill and it doesn't find anything your skill starts playing "The Mourning After" by mac miller
Is it possible to add something like plex or music to the question so that plexmusic only takes the request if it has music in there ?

I tried this skill on an arch linux desktop. It doesn’t install via mycroft-msm, and git cloning the repo and pip installing the requirements, either work

Traceback (most recent call last):
  File "/home/malevolent/development/mycroft-core/mycroft/skills/skill_loader.py", line 204, in _load_skill_source
    skill_module = imp.load_module(
  File "/usr/lib/python3.8/imp.py", line 234, in load_module
    return load_source(name, filename, file)
  File "/usr/lib/python3.8/imp.py", line 171, in load_source
    module = _load(spec)
  File "<frozen importlib._bootstrap>", line 702, in _load
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 779, in exec_module
  File "<frozen importlib._bootstrap_external>", line 916, in get_code
  File "<frozen importlib._bootstrap_external>", line 846, in source_to_code
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/opt/mycroft/skills/plexmusic-skill/__init__.py", line 22
    <<<<<<< Updated upstream
    ^
SyntaxError: invalid syntax
 19:31:24.538 | ERROR    | 217249 | mycroft.skills.skill_loader:_communicate_load_status:287 | Skill plexmusic-skill failed to load

I could not get plexmusic-skill to get the initial data.json from the server. and when it did, I got an error message like this:

'Photo' object has not attribute 'album'

I took a look in the code, and found that the playlist parsing assume all playlists are only music ones (I have a couple of photos playlists).

I commented the playlist parsing section just to try, and plexmusic-skill was immediately able to import the rest of the library.