community[major]: DeepInfra llm and chat #5672
Conversation
lanchain-community [feature]: DeepInfra embeddings integration
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ovuruska
changed the title
ovuruska
changed the title
…o feat/deepinfra-llm-and-chat
dosubot
bot
added
size:L
![dosubot[bot]](https://avatars.githubusercontent.com/in/324583?s=60&v=4){kind=small}
examples/src/llms/deepinfra.ts
Outdated
| @@ -0,0 +1,18 @@ | |||
| import { DeepInfraLLM } from "@langchain/community/llms/deepinfra"; | |||
There was a problem hiding this comment.
Hey team, just a heads up that I've flagged a change in the PR for review. The added code accesses an environment variable using process.env, so it's important to ensure proper handling and security of environment variables.
| @@ -0,0 +1,17 @@ | |||
| import { ChatDeepInfra } from "@langchain/community/chat_models/deepinfra"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code changes, and it looks like the addition of accessing the environment variable DEEPINFRA_API_TOKEN using process.env has been flagged for your review. Please take a look at the related change and ensure it aligns with the project's requirements. Let me know if you need further assistance!
| @@ -0,0 +1,14 @@ | |||
| import { DeepInfraLLM } from "@langchain/community/llms/deepinfra"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code and flagged a change related to accessing environment variables for the maintainers to review. Please take a look at the comment for more details. Let me know if you need further assistance!
| @@ -0,0 +1,217 @@ | |||
| import { | |||
There was a problem hiding this comment.
Hey team, I've reviewed the code and noticed that the new changes introduce a net-new HTTP request using the fetch function. I've flagged this for your review to ensure it aligns with our project's requirements. Great work overall!
| @@ -0,0 +1,217 @@ | |||
| import { | |||
There was a problem hiding this comment.
Hey team, I've reviewed the code changes and flagged a specific section for your attention. The added code appears to explicitly access and require an environment variable using the getEnvironmentVariable function, so please review this change to ensure it aligns with our environment variable handling practices.
| @@ -0,0 +1,69 @@ | |||
| import { LLM, type BaseLLMParams } from "@langchain/core/language_models/llms"; | |||
There was a problem hiding this comment.
Hey team, just a heads up that I've flagged this PR for review because it introduces a new fetch request within the _call method. Please take a look and ensure it aligns with our project's requirements. Thanks!
| @@ -0,0 +1,69 @@ | |||
| import { LLM, type BaseLLMParams } from "@langchain/core/language_models/llms"; | |||
There was a problem hiding this comment.
Hey there! 👋 This is a friendly flag for the maintainers to review the recent change in the PR, as it explicitly requires an environment variable via getEnvironmentVariable. Keep up the great work! 🚀
…feat/deepinfra-llm-and-chat
|
Thank you, sorry for the delay! Have gone through and added some basic docs. |
This PR includes integration of DeepInfra's Inference API with Langchain's LLM and Chat interfaces.