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

Substantial improvements to automatic download process #2

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

eoffermann
Copy link

Substantial refactoring. Using yt_dlp for YouTube downloads instead of youtube-dl.
Added colorama library to improve readability of output.
Improving multiprocessing pooling.
Lots of logging improvements. Implementation of a Logger class.

Minor fix to syntax - the crop filter portion shouldn't have been quoted.
Tweaks to formatting for consistency.
Adding the download directory and main log file to .gitignore

Monumental improvements in internal documentation to make future tweaks to the downloader script more self-explanatory.

张智勐 and others added 21 commits September 13, 2021 10:49
…f youtube-dl. Added colorama library to improve readability of output. Improving multiprocessing pooling. Lots of logging improvements. Implementation of a Logger class.
…f youtube-dl. Added colorama library to improve readability of output. Improving multiprocessing pooling. Lots of logging improvements. Implementation of a Logger class. Substantial additions to have proper docstrings throughout.
…f youtube-dl. Added colorama library to improve readability of output. Improving multiprocessing pooling. Lots of logging improvements. Implementation of a Logger class.
…f youtube-dl. Added colorama library to improve readability of output. Improving multiprocessing pooling. Lots of logging improvements. Implementation of a Logger class. Substantial additions to have proper docstrings throughout.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants