Skip to content
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

automatically save offline #4195

Open
rs4231199 opened this issue Aug 27, 2020 · 5 comments
Open

automatically save offline #4195

rs4231199 opened this issue Aug 27, 2020 · 5 comments
Labels
downloader Issue is related to the downloader feature request Issue is related to a feature in the app

Comments

@rs4231199
Copy link

Describe the feature you want

It is very simple idea but will be very useful. If a video is being watched 2nd or 3rd time, then save it offline so that next time when the same video is being watched it can be played from offline copy.

@rs4231199 rs4231199 added the feature request Issue is related to a feature in the app label Aug 27, 2020
@wb9688
Copy link
Contributor

wb9688 commented Aug 27, 2020

It does already cache streams (up to a certain amount). However the URL to the stream is different every time you'll try to load it, so that cache is not so useful after you restart NewPipe.

@Redirion: Do you happen to know if it's possible for ExoPlayer to use some custom ID (i.e. service name + stream ID + stream ID as introduced by TeamNewPipe/NewPipeExtractor#367) for the cache instead of simply the URL?

@Redirion
Copy link
Member

nice work at your extractor pr @wb9688
In think it could be very useful to work with other attributes than the short living URL. This might also help with some issues where the url became invalid.

However before exploring this option I would like to wait for the imminent release of ExoPlayer 2.12.

Meanwhile could you please re-review #3909?

@wb9688
Copy link
Contributor

wb9688 commented Aug 27, 2020

@Redirion: Yeah, let's wait for ExoPlayer 2.12 before perhaps implementing this feature. We should make an accompanying NewPipe PR for that NewPipeExtractor PR soon though, as it will fix e.g. the long standing #2415, but I hope I'll have time for that this week. I'll review #3909 later today. I'll also make more stream related PRs soon-ish for e.g. not having a hardcoded itag list for YouTube, supporting HDR and 360° videos, etc.

@SameenAhnaf
Copy link
Collaborator

@litetex This issue should be closed in favor of #478 (comment)

@litetex
Copy link
Member

litetex commented Dec 4, 2021

@litetex This issue should be closed in favor of #478 (comment)

Sorry, but I have to counter this as the issue only is about playing videos.

However there is already a caching functionality inside NewPipe so I don't think this is required.
Also downloading all videos when you watch them the 2nd or 3rd (which might be done by accident) time will mess up your available disk space.

So I'm in general against that feature.

@SameenAhnaf SameenAhnaf added the downloader Issue is related to the downloader label Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
downloader Issue is related to the downloader feature request Issue is related to a feature in the app
Projects
None yet
Development

No branches or pull requests

5 participants