-
Notifications
You must be signed in to change notification settings - Fork 454
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
feat: add baichuan llm support #979
feat: add baichuan llm support #979
Conversation
cc @CH3CHO |
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.
由于先合并了另外一个模型的 provider,代码有冲突了。麻烦和上面的评论一并处理一下。谢谢!
Updated comments in provider.go, add "yi"
疑似api token泄露 |
Ⅰ. Describe what this PR did
Support baichuan ai llm, api documentation (https://platform.baichuan-ai.com/docs/api)
Ⅱ. Does this pull request fix one issue?
fix #952
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
envoy.yaml:
Request:
Response:
Ⅴ. Special notes for reviews
在baichuan ai的api文档中,我发现有一个api parameter是
with_search_enhance
, 这是一个bool
类型的参数,每次打开需要消耗额外的token,我认为该参数只需要从context json中获取即可,故没有添加到文档和代码中。