-
Notifications
You must be signed in to change notification settings - Fork 2.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
docs[minor]: Replace deprecated code examples with LCEL & AgentExecutor #6149
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -17,36 +17,29 @@ export const run = async () => { | |||
}), |
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.
Hey there! I noticed that this PR introduces a new external HTTP request to pull the prompt template. I've flagged this change for your review to ensure it aligns with our project's requirements. Let me know if you have any questions or need further clarification!
@@ -1,14 +1,14 @@ | |||
import { LLMChain } from "langchain/chains"; |
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.
Hey team, I've flagged a change in the PR that accesses an environment variable using process.env
. Please review this to ensure it aligns with our security and configuration best practices.
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.
nice! Thanks for doing this
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.
hey looks like the format CI check is failing, could you run yarn format
inside ./examples
and push up those changes files
Sure let me check :)
…On Fri, 19 Jul 2024 at 21:56, Brace Sproul ***@***.***> wrote:
***@***.**** commented on this pull request.
hey looks like the format CI check is failing, could you run yarn format
inside ./examples and push up those changes files
—
Reply to this email directly, view it on GitHub
<#6149 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AU4HPGOUOU4DPL7RMT5CXM3ZNE4ZTAVCNFSM6AAAAABLEEV4XOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCOBYGYZDIOJVGA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Done I have pushed it. |
Description: Added LangChain v0.2 support For the LangChain AgentExecutor using react agent executor which manages a state that is defined by a list of messages in a similar way to the AgentExecutor. Removed classes as well like LLMChain which were deprecated from LangChain v0.1.
Used the LangChain Expression Language (LCEL) instead of LLMChain deprecated class