rickyphewitt / emby-skill

This skill allows audio playback from an emby server
11 stars 19 forks source link

Stop using "instant mix" for the default action #35

Open singpolyma opened 5 years ago

singpolyma commented 5 years ago

Emby's "instant mix" feature is basically useless in my library -- it just plays random songs basically when I use it. So when I say "hey mycroft, play from mb" it just plays random things. The artist and album specifiers work, but it would be nice if only items matching my search term would play in the generic case.

rickyphewitt commented 5 years ago

I've thought about changing this behavior in the past, instead opting to add the specifiers. I don't think just playing items that match the search term would fix this. The reason being is that a search may return artists/albums/songs for the specified term/terms. It is just a search and as general of a search as it gets in the skill. This can result in unexpected results when artists/albums/songs share search term(s). Currently the 1st item is picked in the list and an instant mix started. There are other options if you decide to pick only one:

Or do you look at all the items returned and attempt to create a queue of songs based on the items returned using similar choices as shown above. In that case it would end up being an imitation of an instant mix.

Without the specifiers it is hard to tell the intent of the user which is why the default action is to create an instant mix and let Emby do all the heavy lifting.

Now that being said I have thought about the default action being more interactive. If more than one item is returned MyCroft asks the users to choose an item and play. This appeals to me as a better long term solution and would allow the user to narrow down any results to the point where they play exactly what they intended.