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

Swarm #17

Closed
wants to merge 11 commits into from
Closed

Swarm #17

wants to merge 11 commits into from

Conversation

marklysze
Copy link
Collaborator

@marklysze marklysze commented Nov 14, 2024

Why are these changes needed?

Introducing the ability to create a Swarm of agents in AG2 (based on OpenAI's recent swarm release).

Closed in favour of Swarm V2 - see #30

Related issue number

N/A

Checks

messages: Optional[List[Dict]] = None,
sender: Optional[Agent] = None,
config: Optional[OpenAIWrapper] = None,
) -> Tuple[bool, SwarmResult]:
Copy link
Collaborator

Choose a reason for hiding this comment

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

The return type breaks compatibility with the base class.
Can we avoid it?
If we use nested chat to do tool execution, we won't have this problem.

@marklysze
Copy link
Collaborator Author

Closed - see Swarm V2, #30

@marklysze marklysze closed this Nov 18, 2024
@marklysze marklysze deleted the swarmagent branch November 26, 2024 18:34
davorrunje pushed a commit that referenced this pull request Dec 19, 2024
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.

3 participants