Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

Enable playback of local files #23

Closed
barolo opened this issue Dec 29, 2020 · 11 comments
Closed

Enable playback of local files #23

barolo opened this issue Dec 29, 2020 · 11 comments
Labels
enhancement New feature or request

Comments

@barolo
Copy link

barolo commented Dec 29, 2020

I'm not sure if it would be better to ask upstream, if so just say.
NewPipe has a great 'skip silence' feature which I would love to use with everything but it doesn't even play files which it downloaded itself.

@barolo barolo added the enhancement New feature or request label Dec 29, 2020
@polymorphicshade
Copy link
Owner

@barolo this should be brought up with the original NewPipe devs, since this would be a fairly larger and more "generic" change to the app (and not really SponsorBlock-specific), but I can look in to this just for the fun of it.

@barolo
Copy link
Author

barolo commented Dec 30, 2020

Guessed so, should I close this then? Or leave it for the time being, at least until any relevant info appears?

@polymorphicshade
Copy link
Owner

I'd say leave it open for now. If I decide it's too much work for what it's worth then I'll close it.

@barolo
Copy link
Author

barolo commented Dec 30, 2020

Upstream said no TeamNewPipe#1501 ,kinda

@nadiration
Copy link

Wouldn't recommend this. The app is simple and great as it is bloating it with local videos and other unnecessary features will mess it up. Simplicity is the best and it's rare these days.

@barolo
Copy link
Author

barolo commented Jan 7, 2021

That's a purely subjective opinion, what you consider bloated might not be for others, it already plays videos ffs, most of it is on exo's side too. The app already has support for couple of other services that I've never used.
Your concerns would be better suited for upstream, as this fork's raison d'être is a feature that was deemed unfit for the main one, by its devs.

@barolo
Copy link
Author

barolo commented Jan 14, 2021

@polymorphicshade I've just found that AntennaPod, since latest version, supports playback of local files, including videos, has the silence skip feature [using exo too] and is the perfect companion for NewPipe.
So feel free to close this issue since I don't really feel like it's needed in NP any longer.

@polymorphicshade
Copy link
Owner

I'm back from a small break. I don't plan on closing this for I think a built-in local player would be neat. 😃

@barolo
Copy link
Author

barolo commented Jan 20, 2021

Great :D I mean... it's kind of absurd that NewPipe can't play the files it downloaded itself...

@polymorphicshade
Copy link
Owner

@barolo actually I think there is a good reason for this. Their implementation of the exoplayer seems to be very heavily tied to streaming a remote uri. Simply reusing their implementation for a local uri would result in a lot of domino-effect changes and potentially sloppy workarounds.

It doesn't seem like a simple change... at least based on what I've looked at. This is the nature of software development...

@polymorphicshade
Copy link
Owner

Release 0.20.9 w/ SponsorBlock now includes a built-in player for local playback with SponsorBlock support. Please report any issues you have with it in #30. 😊

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants