-
Notifications
You must be signed in to change notification settings - Fork 27
Add update for langchain example #9
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
base: main
Are you sure you want to change the base?
Add update for langchain example #9
Conversation
| "langgraph>=0.5.4", | ||
| "langchain-openai>=0.3.33", | ||
| "langgraph-cli[inmem]>=0.4.2", | ||
| "langgraph>=0.6.7", |
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.
We have a few conflicts here.
our current auth0-ai-langchain relies on langgraph >=0.5.3, <0.6.0.
our current auth0-fastapi==1.0.0b4 depends on fastapi>=0.115.11,<0.116.0
Also seeing that langgraph-cli[inmem] is also currently confilcting:
And because langgraph-cli[inmem]==0.4.2 depends on langgraph-api>=0.3
and only langgraph-cli[inmem]<=0.4.2 is available, we can conclude that
langgraph-cli[inmem]>=0.4.2 depends on langgraph-api>=0.4.7.
And because your project depends on langgraph-api==0.2.102 and
langgraph-cli[inmem]>=0.4.2, we can conclude that your project's requirements
are unsatisfiable.
Preference here would be to keep current supported versions of those for now, and target this langgraph upgrade with a separate ticket / PR to the auth0-ai-python core first.
| "@langchain/core": "^0.3.75", | ||
| "@langchain/langgraph-sdk": "^0.1.2", |
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.
Not positive here w/ these. We will have to again test after backend langgraph SDK updates though.
|
proposal here would be to hold this PR for future w/ auth0-ai-python alongside updates. I think async auth updates can be covered alongside #15 for now. |
|
Good idea, this will help avoid the same thing as pr: |
Add update for LangChain example