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

Add format config option and improve download reliability #57

Conversation

Owez
Copy link
Owner

@Owez Owez commented Jan 6, 2023

Thanks to @alexislours* for the initial issue and research. This may be extended in Yark 1.3 or beyond with an FFMPEG merging solution to get and merge the highest-quality files from YouTube into a video. For now, this PR serves as a patch to fix issues from yt-dlp/yt-dlp#3372 and add some customization to downloading.

@Owez Owez added bug Something isn't working enhancement New feature or request priority Fast-tracked issue labels Jan 6, 2023
@Owez Owez self-assigned this Jan 6, 2023
@Owez Owez linked an issue Jan 6, 2023 that may be closed by this pull request
@alexanderdavidj
Copy link
Contributor

Credited wrong person 💀

@Owez
Copy link
Owner Author

Owez commented Jan 6, 2023

Credited wrong person 💀

Ahh my bad haha

@alexanderdavidj
Copy link
Contributor

Hah no worries

@Owez
Copy link
Owner Author

Owez commented Jan 7, 2023

The upcoming changes could simplify a lot of this yt-dlp/yt-dlp#4846

@Owez Owez closed this Jan 8, 2023
@Owez Owez force-pushed the 55-use-a-format-fallback-or-allow-to-manually-set-the-video-format branch from d0090ba to 092dee0 Compare January 8, 2023 23:41
@Owez
Copy link
Owner Author

Owez commented Jan 8, 2023

Sorry about the mess, removing changes and resetting to current origin/master

@Owez
Copy link
Owner Author

Owez commented Jan 9, 2023

New --format option implemented with a new message if ffmpeg isn't installed:

image

I don't want to try to download 480/360p quality videos even if its a possible mitigation. Might add a warning to say "FFMPEG is reccomended" if it's not installed at startup (and add docs to README) to solve #59 because the format quality issues have been resolved here.

@Owez
Copy link
Owner Author

Owez commented Jan 9, 2023

Superceded by #66

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request priority Fast-tracked issue
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Use a format fallback or allow to manually set the video format
2 participants