Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update better downloader and audio convertion & summarization #10

Merged
merged 7 commits into from
Nov 23, 2023
Merged

Update better downloader and audio convertion & summarization #10

merged 7 commits into from
Nov 23, 2023

Conversation

cobanov
Copy link
Contributor

@cobanov cobanov commented Nov 23, 2023

The mp4 files streamed from pytube had to be converted to mp3 format with the correct codec, and a small adjustment was made to the file naming. If the video does not download, a more descriptive error message is provided. At the same time, simple logging messages are written to the screen for the download process.

In the demo in the readme file, the return command instead of print was forgotten and the attribute in the language option wasn't closed correctly.

@cobanov cobanov changed the title Update better downloader and audio convertion & readme.md Update better downloader and audio convertion & summarization Nov 23, 2023
@cobanov
Copy link
Contributor Author

cobanov commented Nov 23, 2023

I'm sorry for committing the text summerization commits on the same pull request.

@kadirnar kadirnar added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 23, 2023
@kadirnar
Copy link
Owner

I'm sorry for committing the text summerization commits on the same pull request.

Thank you for your contribution. I tested the model and the result is very good 🚀

@kadirnar kadirnar merged commit 65ef57e into kadirnar:main Nov 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request] Add Text Summarization with Open Source LLM Models 📚
2 participants