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

rename eb-turbo-appbuilder #177

Merged
merged 9 commits into from
Mar 12, 2024
Merged

Conversation

MrChengmo
Copy link
Collaborator

@MrChengmo MrChengmo commented Mar 11, 2024

重命名模型 eb-turbo-appbuider 为 ERNIE Speed-AppBuilder

README.md Outdated
@@ -71,7 +71,7 @@ print(", ".join(models))

填写自己的Token,获取模型列表输出如下:
``` shell
ERNIE-Bot 4.0, ERNIE-Bot-8K, ERNIE-Bot, ERNIE-Bot-turbo, EB-turbo-AppBuilder专用版, Qianfan-Chinese-Llama-2-7B, Yi-34B-Chat, Llama-2-7B-Chat, Llama-2-13B-Chat, Llama-2-70B-Chat, ChatGLM2-6B-32K, ChatLaw, BLOOMZ-7B, Qianfan-BLOOMZ-7B-compressed, AquilaChat-7B
ERNIE-Bot 4.0, ERNIE-Bot-8K, ERNIE-Bot, ERNIE-Bot-turbo, ERNIE Speed-AppBuilder, Qianfan-Chinese-Llama-2-7B, Yi-34B-Chat, Llama-2-7B-Chat, Llama-2-13B-Chat, Llama-2-70B-Chat, ChatGLM2-6B-32K, ChatLaw, BLOOMZ-7B, Qianfan-BLOOMZ-7B-compressed, AquilaChat-7B
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

当前获取模型列表输出的名字应该是"EB-turbo-AppBuilder专用版", 确认下名字是否需要转成"ERNIE Speed-AppBuilder",这样使用新版本SDK的用户看到的就是"ERNIE Speed-AppBuilder"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Readme中展示的是模型原始名,所以此处输出的列表为EB-turbo-AppBuilder专用版,该问题已修复,感谢。
  2. 已经在get_model_listfunc处添加todo,使用appbuilder.get_model_list()应当返回一个包含原始名及相应short名的列表,该功能会在其他PR提供

@ide-rea
Copy link
Contributor

ide-rea commented Mar 12, 2024

LGTM

@MrChengmo MrChengmo merged commit 580b00d into baidubce:master Mar 12, 2024
1 check failed
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.

2 participants