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

feat: add Higress ai model provider #4755

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

WindWaving
Copy link

💻 变更类型 | Change Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 👷 build
  • ⚡️ perf
  • 📝 docs
  • 🔨 chore

🔀 变更说明 | Description of Change

设置-语言模型新增支持Higress AI provider

📝 补充信息 | Additional Information

语言模型新增Higress provider

image

对话信息

image
image

Copy link

vercel bot commented Nov 22, 2024

Someone is attempting to deploy a commit to the LobeChat Community Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. Model Provider 模型服务商 🌠 Feature Request New feature or request | 特性与建议 labels Nov 22, 2024
@lobehubbot
Copy link
Member

👍 @WindWaving

Thank you for raising your pull request and contributing to our Community
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.
If you encounter any problems, please feel free to connect with us.
非常感谢您提出拉取请求并为我们的社区做出贡献,请确保您已经遵循了我们的贡献指南,我们会尽快审查它。
如果您遇到任何问题,请随时与我们联系。

Copy link

codecov bot commented Nov 24, 2024

Codecov Report

Attention: Patch coverage is 99.10471% with 23 lines in your changes missing coverage. Please review.

Project coverage is 92.01%. Comparing base (3da1704) to head (28e3dd1).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/libs/agent-runtime/higress/index.ts 55.55% 20 Missing ⚠️
src/libs/agent-runtime/AgentRuntime.ts 57.14% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4755      +/-   ##
==========================================
+ Coverage   91.53%   92.01%   +0.48%     
==========================================
  Files         538      540       +2     
  Lines       37803    40369    +2566     
  Branches     3651     2327    -1324     
==========================================
+ Hits        34603    37146    +2543     
- Misses       3200     3223      +23     
Flag Coverage Δ
app 92.01% <99.10%> (+0.48%) ⬆️
server 96.21% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@arvinxx arvinxx left a comment

Choose a reason for hiding this comment

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

Rebase 下 main,lodash 那个改下,其他没问题了~

src/libs/agent-runtime/higress/index.ts Outdated Show resolved Hide resolved
src/config/llm.ts Outdated Show resolved Hide resolved
@WindWaving
Copy link
Author

Rebase 下 main,lodash 那个改下,其他没问题了~

已经改好了🙏

@lobehubbot
Copy link
Member

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Rebase main and lodash, and the rest will be fine~

Already changed🙏

Copy link
Contributor

@arvinxx arvinxx left a comment

Choose a reason for hiding this comment

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

两个 README 的变动撤销下?不应该带在这个 PR 里的

Copy link
Contributor

Choose a reason for hiding this comment

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

这一段应该也不需要了,Higress 的配置应该和默认的一样, 走 default 那一段就好

Copy link
Author

Choose a reason for hiding this comment

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

done🌹

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌠 Feature Request New feature or request | 特性与建议 Model Provider 模型服务商 size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants