Use maximum available resolution for thumbnail#835
Use maximum available resolution for thumbnail#835ritiek wants to merge 1 commit intomps-youtube:developfrom
Conversation
|
Perhaps change this to call the method from pafy? That means it pulls in pafy's |
|
Ok, I've changed it to use Also, I think we should release a new version of pafy before merging this otherwise we'll get half the issues complaining about |
|
Let's just remove the pafy internal backend. I don't think anyone uses it anymore. |
|
Actually, it still lags (about 3s) probably because of |
This PR improves thumbnail quality in metadata when using mpris API (instead of always falling back to
default.jpg).One drawback is that this makes things a little slower (checking if URLs responds with 200 or not). Not sure if it is worth it though, let's decide!See #835 (comment).Cross reference: mps-youtube/pafy#211