Add seek support for youtube videos

[pmb6tz]

Hey, thanks for the great work on this plugin!

It would be awesome if we could seek forward/backward in the videos. I know this is possible in mplayer w/ffmpeg using the same stream url that the plugin uses, so I wondered if it would be possible here.

If you want to point me in the right direction I could see about doing this myself and submitting a pull request. I mainly don't quite understand how to override the playback functionality myself.

[lukasmrtvy]

+1

[BrainStone]

Is this an issue with Mopidy or with Mopidy-YouTube?

P.S.: I'm dumb. I thought this was on the Mopidy repo, not the Mopidy-YouTube repo.

[natumbri]

I'm thinking about adding file caching, which passes a file on a disk to gstreamer. Using a cached file, rather than the file direct from YouTube might solve any seeking issues.

New branch here: feature/cache

Config option required:

allow_cache = true

Feedback appreciated.

[natumbri]

@pmb6tz @lukasmrtvy @BrainStone is anyone interested in testing this? I've merged into develop. Would be interested to hear if it helps, or if you're no longer interested (in which case I'll close the issue).

Cheers.

[BrainStone]

I currently don't use this software anymore. But if I would I'd still be interested, fyi.

[kingosticks]

Did you ever try https://docs.mopidy.com/en/latest/api/backend/?highlight=should_download#mopidy.backend.PlaybackProvider.should_download for this @natumbri ?

[natumbri]

That's a good question @kingosticks! Somehow it ended up in the PlaybackProvider but I'm not convinced that I ever tested it, or even thought carefully about whether it is a good idea for this extension.

[kingosticks]

I would have thought it'd make an impact (good or bad) for an issue like this.

[natumbri]

Yeah, absolutely it seems relevant. I will have a look at it - thanks for raising.