Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added support to download audio output only #268

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

Merged
DhanushNehru merged 1 commit into DhanushNehru:master from shailesh-vaidya:yt_download
Oct 2, 2024

Conversation

@shailesh-vaidya
Copy link

@shailesh-vaidya shailesh-vaidya commented Sep 11, 2024
edited
Loading

Added support to download audio output only

Summary

First of all thanks for providing awesone python scripts.
Sometime we need to download youtube video as audio files only. I have added that support in youtubeDownloader.py. I also observed "UnicodeEncodeError" error for few videos. Added fix for that also.

Description

Added option to download only audio output of youtube video

The changes are as follows:

  • Modified code to configure ydl_opts['format'] option to only audio or audio+video based on input provide
  • Observed "UnicodeEncodeError: 'charmap' codec can't encode character '\uff5c' in position 86: character maps to <undefined>" error while downloading few of the videos. Added 'restrictfilenames': 'True', option to fix this. Refer - Critical: | in yt videos throws an error yt-dlp/yt-dlp#5819

Checks

Validated downloads with multiple videos.

in the repository

  • Made no changes that degrades the functioning of the repository
  • Gave each commit a better title (unlike updated README.md)

in the PR

  • Followed the format of the pull_request_template
  • Made the Pull Request in a small level (for the creator's wellfare)
  • Tested the changes you made

Thank You,

Shailesh Vaidya

Signed-off-by: shailesh-vaidya <Shailesh Vaidya>
@shailesh-vaidya shailesh-vaidya marked this pull request as ready for review September 11, 2024 20:16
@DhanushNehru DhanushNehru merged commit 897346b into DhanushNehru:master Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DhanushNehru DhanushNehru DhanushNehru approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /