-
Notifications
You must be signed in to change notification settings - Fork 87
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
Langgraph integration #52
base: main
Are you sure you want to change the base?
Conversation
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.
@s-udhaya can the supervisor be a langgraph model? I think in practice this is how most users with LangGraph code will expect to integrate.
@smurching @niall-turbitt wdyt?
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.
@FMurray Multiagent implementation for sure can be a langgraph model. This one is a direct copy from the opensdk implementation and multiagent was not part of the initial implementation. I am now in the process of cleaning this up. Will convert this PR to draft.
@FMurray I have removed the unused modules and simplified the configurations. While I'm working on the multi-agent functionality, could you please do a quick review of the PR? Thanks! |
Related Issues/PRs
NoneWhat changes are proposed in this pull request?
Implement function calling agent using langgraph. Tested with UC tool and vector search retriever tool
How is this PR tested?