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

Add "Local Media Files" tab in drawer once the media player is stable #52

Closed
3 tasks done
SameenAhnaf opened this issue Mar 25, 2021 · 13 comments
Closed
3 tasks done
Labels
enhancement New feature or request

Comments

@SameenAhnaf
Copy link

Checklist

Describe the feature you want

Newpipe has an amazing media player built-in. Unfortunately, we can't play other media files with this player. I wish to have "Local Media Files" tab in drawer once the media player is stable. Plus, this tab should have separate drawers for "Video" and "Audio" sections to avoid mix-up.

Is your feature request related to a problem? Please describe it

Additional context

How will you/everyone benefit from this feature?

@SameenAhnaf SameenAhnaf added the enhancement New feature or request label Mar 25, 2021
@polymorphicshade
Copy link
Owner

I won't be doing this. There are plenty of other apps designed to be local media players.

@SameenAhnaf
Copy link
Author

@polymorphicshade Sorry for the disturbance. But, here are my few words to say.

I respect your time, efforts and dedication to this app. Staying up-to-date with an app bringing necessary features is a really tough job. But would you please reconsider adding this tab? I don't need support for all file types. Only, Newpipe supported (for download and streaming) formats.

Thanks in advance for your dedication!

@barolo
Copy link

barolo commented Sep 9, 2021

I won't be doing this. There are plenty of other apps designed to be local media players.

Since it already plays downloaded files would it be hard to expose the player to the system [ I might have asked it already somewhere but cannot find it now ].
There's nothing else that has such a great implementation of skip silence unfortunately.

@Atemu
Copy link

Atemu commented Sep 10, 2021

This is a great idea and something I've been wanting aswell but, why are you asking for it here? This fork is concerned with SponsorBlock and not much else.

Go suggest this feature upstream. We have all the features they have + SponsorBlock + some small extras.

When this has been implemented upstream, we can think about SponsorBlock-specific adaptations, i.e. sponsor skipping in downloaded videos if they come from youtube.

Feel free to ping me in the upstream issue.

@SameenAhnaf
Copy link
Author

@Atemu What should I do? I was actually suggested to use Sponsorblock fork as stated in TeamNewPipe#5666 (comment) ? Will I reopen that issue?

@Atemu
Copy link

Atemu commented Sep 10, 2021

@SameenAhnaf, @MD77MD didn't mean that you should close the issue, I think they meant that poly had implemented something similar before and might be able to lend a hand in implementing it upstream.

This fork only exists because upstream explicitly decided against having SponsorBlock functionality and there's nothing indicating that the same is true for this feature. Just re-open the upstream issue.

@barolo
Copy link

barolo commented Sep 10, 2021

This is a great idea and something I've been wanting aswell but, why are you asking for it here? This fork is concerned with SponsorBlock and not much else.

Go suggest this feature upstream. We have all the features they have + SponsorBlock + some small extras.

When this has been implemented upstream, we can think about SponsorBlock-specific adaptations, i.e. sponsor skipping in downloaded videos if they come from youtube.

Feel free to ping me in the upstream issue.

If I'm recalling correctly, current ability of this fork to play downloaded files stemmed from my initial request of playing local files which got rejected upstream.
Why not go extra step and just enable externals? [I have answered my own question, the implementation of player is heavily tied to streaming, and enabling playing of local files apparently is non trivial]

[Edit. Yup, here it is https://github.com//issues/23 ]

@Atemu
Copy link

Atemu commented Sep 15, 2021

Your link is broken.

@barolo
Copy link

barolo commented Sep 15, 2021

Your link is broken.

It is not, but just in case: https://github.com/polymorphicshade/NewPipe/issues/23

@SameenAhnaf
Copy link
Author

@barolo TeamNewPipe#5666 is already open. No need to talk about this in this repository any more. Mainstream Newpipe has more users. So, it should atteact most devs' attention.

@barolo
Copy link

barolo commented Sep 16, 2021

@barolo TeamNewPipe#5666 is already open. No need to talk about this in this repository any more. Mainstream Newpipe has more users. So, it should atteact most devs' attention.

Thank you for letting me know, it was created after another such request got closed/rejected by one of the devs.

Edit. You closed and reopened the linked issue multiple times, also the devs upstream indicated that they're not interested in such feature.

@barolo
Copy link

barolo commented Sep 16, 2021

JustPlayer got skip-silence implemented recently so I officially don't care any-longer, it's not as good [doesn't fast-forward just cuts] but it's good enough for me.
Hopefully will be improved over time.

@iamashwin99
Copy link

iamashwin99 commented Apr 17, 2022

for those who really need the video player to work, here's a trick to get a specific file to appear in the downloads section if you have root permission.
Open the file /data/data/org.polymorphicshade.newpipe/databases/downloads.db in a SQL editor( I suggest MiXplorer) add a new entry with the path to the video file, a random youtube URL, a random byte count, a random epoch time stamp, and the kind as 'v' for video and 'a' for audio, set the segments as NULL. (See the image below for a better idea).

screenshot

Now save the database, and restart the app. and you should see the local video in the download section,

the down side of this method is that you have to change the video path manually, and you miss out on fast forward, brightness control, speed control etc.. I would still think that it would be easier to implement a "share" or "open with" dialogue for the app so that we could use some local explorer and "share" the video to newpipe. This would be easier to implement than adding a tab in the drawer and would solve most of the requirements of users of this thread. Although Im no app developer, so not sure if my guess is correct.

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

5 participants