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

Feature Request: Pull video in HD #3

Open
meinnamehier opened this issue Mar 5, 2016 · 5 comments
Open

Feature Request: Pull video in HD #3

meinnamehier opened this issue Mar 5, 2016 · 5 comments

Comments

@meinnamehier
Copy link

When a higher resolution video is available, it should be pulled
E.g. http://www.arte.tv/guide/de/056821-002-A/360-geo-reportage
was not pulled in HD, but it is available in HD.
Or better yet: Make the resolution an option, such as --resolution highest or so

@sboehringer
Copy link
Owner

I tried to implement the feature. This script relies on information from the ZDFmediathek project. The raw database pulled from said project can be found in

 ~/.local/share/applications/mediathek/database-json.xz

If you can figure out a way to reliably extract (working) HD-urls in case they are given, I would be happy to implement the feature. Back when I tried the urls in the file often did not work.

@meinnamehier
Copy link
Author

I gues that is a little out of my league, but there maybe https://github.com/iNaD can help.
He wrote download hosts for Synology DSM which work mostly. If i get around to it, I will contact him (or you can, if you wish.)

@Hinterwaeldlers
Copy link

Using the pull request #11 as base allows to implement this feature.

Using this part as base, it should be possible, to do the following:
--highest
--lowest
--medium
So it's free to the user, to select which variant to use.
Sadly I've not found the part yet, where such a part could be appended to. Sadly I am not familiar with perl, so feel free to adept those changes within the pull request to implement this feature

@sboehringer
Copy link
Owner

Thank you for the pull request. I have that parses raw input into the parse.pl tools. Could you move that code over? Then I will merge and also implement a way to select the appropriate field.

@Hinterwaeldlers
Copy link

Sorry but I am not sure, which file is meant. The main change was done in parse-videolist-json.pl and I don't see parse.pl file, so not sure, if the file already fits. Otherwise, please let me know and I can adept the change accordingly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants