-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Comments
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? |
nice work at your extractor pr @wb9688 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? |
@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. |
@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. So I'm in general against that feature. |
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.
The text was updated successfully, but these errors were encountered: