-
Notifications
You must be signed in to change notification settings - Fork 7
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
General_playback_error #12
Comments
@matjaz If it helps here is a track that will reproduce the error without fail: Search for 'naturally'
The Youtube app returns an error when Homey asks for a stream_url: |
@MarvinSchenkel can you please take a look? |
Looks like you have to setup your own api key. Go to settings to register one. I am looking into implementing logging in to YouTube so we won't encounter this error anymore |
@MarvinSchenkel I have a own api key so with this link it works with https://www.youtube.com/watch?v=f4Mc-NYPHaQ&ytbChannel=Queen%20Official but it does not work with the first one if you type in queen in the search of "Muziek" and try to play it you get generic_playback_issue which is this one |
@chrisboer Hm, starting to look like it has to do with long videos (> 1h). Can you confirm you get the error with relative long videos? |
Yeah figured it out. It has to do with the youtube>mp3 library we use to create the stream url. I get the following error if I manually try to convery the video on their website: There was an Error caused by YouTube, we cannot deliver this Video! @matjaz Any idea for a better youtube>mp3 converter? |
Yes with long videos but also with commercial video's (short) like if you want to play "naturally" from the artist Ronde you also get a generic_playback_error |
This one looks good http://www.youtubeinmp3.com/api/ |
@matjaz Implemented this one. Looks like this one has a 120 minutes limit, should be alright for most videos. I have look for other ones, but could not find one that converts videos that are longer than 120 minutes. Looks like Homey does not pick up any errors thrown by the app, but it always displays 'generic_playback_error'. |
@MarvinSchenkel A user friendly approach would be to filter any tracks that exceed this limit when responding to media search queries. |
@TheMSB Thanks, really good idea. Fixed it and implemented the filter. The duration is now also shown in the search results :). Will be committed as part of the next release. I am still working on the login so I can fetch personal playlists. |
filter is necessary only for local mp3 playback (requiring converting service). chromecast and others which stream video directly can handle longer videos. |
Exactly, it would make sense to implement this filter only when responding to search/play requests made by Homey media. |
Of course I only applied the filter for homey media playback ;) |
version ff345b7 should fix this |
https://github.com/athombv/homey/issues/1393
The text was updated successfully, but these errors were encountered: