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

[Feature]: 允许基础功能指定模型 #1708

Merged

Conversation

awwaawwa
Copy link
Contributor

类似 deepseek-math-7b-rl 的模型解决领域特定问题能力比较强,但是平时正常日用的肯定不用它,只会有少数功能使用该模型。所以如果基础功能能指定模型就无需手动切来切去了,省点事。

@binary-husky
Copy link
Owner

非常有用的建议,我修改了一下参数名字,烦请有空时测试下有没有BUG

@awwaawwa
Copy link
Contributor Author

当前代码在 bridge_all.py 的第950和951行初始化 functional,这样会导致模型重载不受热更新影响。所以我又把那几行代码挪回去了......。另外补了个报错,功能测试正常。

@binary-husky binary-husky merged commit b7eb9ab into binary-husky:master May 17, 2024
1 check passed
@awwaawwa awwaawwa deleted the allow_core_func_specify_model branch May 17, 2024 10:21
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.

None yet

2 participants