-
Notifications
You must be signed in to change notification settings - Fork 775
Youtube Extractor Fix with NewPipe #2489
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
base: master
Are you sure you want to change the base?
Conversation
fire-light42
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice pull request, tested it and works well. I only have a comment about the quality cap.
library/src/commonMain/kotlin/com/lagradost/cloudstream3/extractors/YoutubeExtractor.kt
Outdated
Show resolved
Hide resolved
|
Sorry, for all this forced-pushed, i wanted to created a seprate create pr for the extractor, now |
fire-light42
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good, will merge after testing.
1. Bump newpipeextractor
File:
gradle/libs.versions.toml2. YoutubeExtractor Fix
Supports up to 4K.
Testing Mr. Beast's Video 4K Fetched
Screen_recording_20260210_042739.webm
Known Issue - Audio Label's not assiging properly
this mostly happens to additional dubbed audios
i already have some fix, will create another pr for it as it's related to how
newAudioFileis handled