openai[minor]: Cherry pick OpenAI changes into 0.1 #5350
Conversation
* Update OpenAI with Azure Specific Code * Export Embeddings * Add deployment parameters * Check to fix build issue * Update libs/langchain-openai/src/azure/embeddings.ts Co-authored-by: Brace Sproul <braceasproul@gmail.com> * Update libs/langchain-openai/src/azure/embeddings.ts Co-authored-by: Brace Sproul <braceasproul@gmail.com> * nit changes * Update Test Descriptions * Format * Fix types --------- Co-authored-by: Brace Sproul <braceasproul@gmail.com> Co-authored-by: jacoblee93 <jacoblee93@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
the
size:XXL
|
Release to follow |
dosubot
bot
added
the
auto:improvement
![dosubot[bot]](https://avatars.githubusercontent.com/in/324583?s=60&v=4){kind=small}
| @@ -1291,7 +1291,7 @@ | |||
| "node-llama-cpp": "2.7.3", | |||
There was a problem hiding this comment.
Hey there! 👋 Just wanted to flag that the change in the "openai" package version from "^4.32.1" to "^4.41.1" might impact the project's dependencies. This is for the maintainers to review and ensure it aligns with the intended dependency type (peer/dev/hard). Keep up the great work!
| @@ -41,11 +41,12 @@ | |||
| "dependencies": { | |||
There was a problem hiding this comment.
Hey there! 👋 I noticed that the package.json file has been updated with changes to peer and dev dependencies, including an update to the "openai" peer dependency and the addition of a new dev dependency "@azure/identity". I'm flagging this for your review. Keep up the great work! 🚀
| @@ -48,6 +43,57 @@ export class AzureChatOpenAI extends ChatOpenAI { | |||
| super(newFields); | |||
There was a problem hiding this comment.
Hey team, I've flagged this PR for your review as it includes changes related to accessing environment variables for the Azure OpenAI API configuration. Please take a look to ensure the handling of environment variables aligns with best practices.
| @@ -0,0 +1,98 @@ | |||
| import { | |||
There was a problem hiding this comment.
Hey team, I've flagged this PR for your review as it introduces a new external HTTP request using the AzureOpenAIClient in the embeddingWithRetry method. Please take a look and ensure it aligns with our project's requirements.
| @@ -0,0 +1,829 @@ | |||
| import { test, jest, expect } from "@jest/globals"; | |||
There was a problem hiding this comment.
Hey team, I've reviewed the PR and noticed that it explicitly accesses and requires environment variables via getEnvironmentVariable. This comment is to flag this change for your review. Let me know if you need any further assistance with this.
| @@ -0,0 +1,333 @@ | |||
| import { test, expect } from "@jest/globals"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code changes and noticed that the addition of environment variable access via getEnvironmentVariable function. I've flagged this for your review to ensure proper handling of environment variables. Let me know if you need further assistance with this.
jacoblee93
changed the title
Cherry pick OpenAI changes into 0.1
@sarangan12 @bracesproul