-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
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
♻️ refactor: Refactor Azure OpenAI Implementation #4619
base: main
Are you sure you want to change the base?
Conversation
Rename apikey to apiKey Refactor o1model check Add generic type for OpenAI.transformResponseToStream
@BrandonStudio is attempting to deploy a commit to the LobeChat Community Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4619 +/- ##
==========================================
+ Coverage 92.43% 92.52% +0.09%
==========================================
Files 509 512 +3
Lines 36602 36820 +218
Branches 2229 3401 +1172
==========================================
+ Hits 33832 34069 +237
+ Misses 2770 2751 -19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
I'm not going to complete TODOs for now. I'd rather wait for refactor of |
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.
如果都换成了下划线模式,应该可以直接用 OpenAI 的 stream处理方法,不需要再单独写一个 Azure Openai 的了?
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.
我记得有一块不一样来着,忘了是哪块了
回头试试
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
@azure/openai
toopenai
apikey
toapiKey
o1Models.has
toisO1Model
OpenAI.transformResponseToStream
📝 补充信息 | Additional Information
Note that on non-database deployment, sending large image results in 431 Request Header Fields Too Large
Same issue happens in GitHub provider
refs:
TODOs: