-
-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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. |
Thank you for raising your pull request and contributing to our Community |
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 |
💻 变更类型 | 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: