-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Support static context in integration with langchain #4504
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
Conversation
392139e to
b3226d6
Compare
This is used to store data about an agent invocation which should be accessible to tools but not to the LLM, like API keys, user IDs etc. See https://docs.langchain.com/oss/python/langchain/context-engineering#runtime-context-6
b3226d6 to
3c8d316
Compare
|
this looks good, could you roll back the uv.lock change and fix the types CI please? |
I cannot roll back the uv.lock change, because this feature was introduced in langgraph 0.6.0. I am trying to fix the type error, it's not clear to me why it's occurring right now. The dependency upgrade should have fixed it. The type error raised by mypy is wrong. I am not really a python developer, so I'd appreciate any help in figuring out why it might be happening. |
Make LLMAdapter and LangGraphStream generic over ContextT and type PregelProtocol accordingly so mypy accepts passing context into astream.
|
@benlangfeld This should fix it: powerhome#3 |
fix(langchain): parameterize langgraph context type
|
@davidzhao This should be ready to go. |
davidzhao
left a comment
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.
lg!
This is used to store data about an agent invocation which should be accessible to tools but not to the LLM, like API keys, user IDs etc.
See:
config['configurable']andconfig_schema) langchain-ai/langgraph#5243