-
Notifications
You must be signed in to change notification settings - Fork 179
add one key download all models #232
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
Open
nickchen110
wants to merge
12
commits into
lenML:main
Choose a base branch
from
nickchen110:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
感谢贡献 @nickchen110 麻烦添加更多信息介绍一下你的想法和修改
|
OK,那就按照你的方式去做修改。感谢回复!
…Sent from my iPhone
On May 15, 2025, at 13:10, len ***@***.***> wrote:
[https://avatars.githubusercontent.com/u/37396659?s=20&v=4]zhzLuke96 left a comment (lenML/Speech-AI-Forge#232)<#232 (comment)>
感谢贡献
@nickchen110<https://github.com/nickchen110> 麻烦添加更多信息介绍一下你的想法和修改
因为,本来是准备写一键下载的,但是模型太多了,几乎没用人需要下载所有模型,所以没搞
我希望的话是可以统一下载的脚本入口,类似 python -m scripts.downloader "chat-tts,cosyvoice,f5tts" 类似这样的脚本选择性的下载
你觉得如何?
* 这个pr中涉及的一些文件和最近修改可能有冲突需要你先合并解决下
* 关于 matcha-tts 的问题已经解决,它的包依赖有冲突,我已经直接引入到 repos_static 下面了
* required_files 最好不要改,因为一般情况都是 modelscope 搬运的时候写错了,所以我们应该改库,或者得改 downloader 的逻辑,不然 hf 里面的库不兼容,我们的代码也不兼容,因为文件名在代码里一般是写死有用的
—
Reply to this email directly, view it on GitHub<#232 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BRZ4HX7D57DV3JJN4KPMOLL26QOSZAVCNFSM6AAAAAB4SYDEPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQOBSGQYDKOBZHE>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
add one key download all models