Skip to content
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

Support function_call in autogen/agent #1091

Merged
merged 47 commits into from
Jul 5, 2023
Merged

Support function_call in autogen/agent #1091

merged 47 commits into from
Jul 5, 2023

Conversation

yiranwu0
Copy link
Collaborator

@yiranwu0 yiranwu0 commented Jun 24, 2023

Why are these changes needed?

Preliminary designs to add the new function_call feature into the agent framework.

Related issue number

See #1083 for tasks.

Checks

Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge with main and simplify the notebook.
add test.

flaml/autogen/agent/user_proxy_agent.py Outdated Show resolved Hide resolved
@yiranwu0
Copy link
Collaborator Author

Some Notes:

  1. In UserProxyAgent.receive, I added back the line to convert the message to a dict. This is to make the type of the message consistent to avoid keep checking the type. Thus, We may define is_termination_msg to always take in a dict. The type of message passed to auto_reply should also be a dict.

  2. UserProxyAgent now allows the function calls calling to a class method, and passing in predefined arguments, or even update these arguments if needed. The current design is to ask the user to pass in a dict. More details in documentation of UserProxyAgent.

@qingyun-wu qingyun-wu temporarily deployed to openai July 5, 2023 04:41 — with GitHub Actions Inactive
@qingyun-wu qingyun-wu temporarily deployed to openai July 5, 2023 04:41 — with GitHub Actions Inactive
@yiranwu0 yiranwu0 temporarily deployed to openai July 5, 2023 09:40 — with GitHub Actions Inactive
@yiranwu0 yiranwu0 temporarily deployed to openai July 5, 2023 14:57 — with GitHub Actions Inactive
@yiranwu0 yiranwu0 temporarily deployed to openai July 5, 2023 14:57 — with GitHub Actions Inactive
@yiranwu0 yiranwu0 temporarily deployed to openai July 5, 2023 14:57 — with GitHub Actions Inactive
@sonichi sonichi added this pull request to the merge queue Jul 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jul 5, 2023
@qingyun-wu qingyun-wu added this pull request to the merge queue Jul 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 5, 2023
@sonichi sonichi added this pull request to the merge queue Jul 5, 2023
Merged via the queue into main with commit ca10b28 Jul 5, 2023
16 checks passed
@sonichi sonichi deleted the funccall branch July 6, 2023 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support function_call in auto_reply of UserProxyAgent
5 participants