-
Notifications
You must be signed in to change notification settings - Fork 327
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
Fix:readme #225
Fix:readme #225
Conversation
多轮调用时候的完整prompt和模型输出都打印下? |
README.md
Outdated
# instantiation LLM | ||
model_name = 'modelscope-agent-7b' | ||
llm = LLMFactory.build_llm(model_name, model_cfg) | ||
model_name = 'qwen-max' |
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.
最好使用qwen-72b,开源模型
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.
已修改成72b,并已验证效果ok
@@ -44,46 +44,46 @@ | |||
"is_active": true, | |||
"use": false | |||
}, | |||
"modelscope_text-address": { |
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.
why remove modelscope?
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.
之前模型调用效果差,尝试把工具中的前缀modelscope_删除以避免信息干扰
@@ -33,8 +33,9 @@ def _local_parse_input(self, *args, **kwargs): | |||
def _remote_parse_input(self, *args, **kwargs): | |||
if 'gender' not in kwargs: | |||
kwargs['gender'] = 'man' | |||
voice = 'zhizhe_emo' if kwargs['gender'] == 'man' else 'zhiyan_emo' | |||
kwargs['voice'] = voice | |||
voice = 'zhizhe_emo' if kwargs['gender'] == 'man' or kwargs[ |
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.
fix readme 基于(依赖)#220
具体原因:readme中case使用的视频生成等tool 在pr 220中被修复。
@@ -44,46 +44,46 @@ | |||
"is_active": true, | |||
"use": false | |||
}, | |||
"modelscope_text-address": { |
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.
之前模型调用效果差,尝试把工具中的前缀modelscope_删除以避免信息干扰
README.md
Outdated
# instantiation LLM | ||
model_name = 'modelscope-agent-7b' | ||
llm = LLMFactory.build_llm(model_name, model_cfg) | ||
model_name = 'qwen-max' |
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.
已修改成72b,并已验证效果ok
readme可运行,运行效果:
单轮:
多轮