forslund / mopidy_skill

A Mycroft skill for controlling the mopidy media player
GNU General Public License v3.0
15 stars 14 forks source link

Use Mopidy API directly #16

Open ndarilek opened 6 years ago

ndarilek commented 6 years ago

I'm struggling a bit with some aspects of this as implemented:

Would you accept a PR that does the following?

I'm happy to do the work, I just figure there's a reason you scrape the library and create intents manually. Maybe it's so fuzzy text searches can work, but if so I wonder how much of a performance hit it is to fetch the data each time and search that?

Thanks.

forslund commented 6 years ago

Hi,

I'd be happy to accept something like that. This skill was basically the first skill I made for mycroft and it uses Adapt in a way that makes the play intents very restrictive. I.e the artist/album/genre is scraped and generates vocab. Which is good in theory but makes it a bit tricky to activate. It's worked ok for me but I have don't have a very large library.

I've have started on a couple of much needed updates to the skill but no major overhaul as you suggest.

Looking forward to your PR. Let me know if you need to bounce any ideas or if you want to split up the work.

ndarilek commented 6 years ago

Hmm, that was short-lived. Mopidy doesn't run under Python 3. Neither does python-mpd. So it looks like we'll be using the custom client for now.

I'll see what I can do with the existing client library and a few intents with slots for album/artist/track.