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

Fix one failure mode in the user guide #4658

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

peterychang
Copy link
Collaborator

@peterychang peterychang commented Dec 11, 2024

Why are these changes needed?

The user guide will often fail because the LLM does not provide all the code required to solve the problem in a single response

Related issue number

Resolves #4645

Checks

@peterychang
Copy link
Collaborator Author

This PR has a tab spacing diff. I can try to fix it if it matters, but if not, use this to see the actual change

https://github.com/microsoft/autogen/pull/4658/files?w=1

(just append ?w=1 to the diff URL to ignore whitespace)

Copy link
Collaborator

@ekzhu ekzhu left a comment

Choose a reason for hiding this comment

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

Let's merge this change.

However I think this quick start isn't so quick. We should consider moving this to the multi-agent design pattern section, and update this quick start with something super simple, such as two agents counting from 10 to 1, without using any model or executor, just pure Python.

@jackgerrits jackgerrits merged commit a65855d into microsoft:main Dec 12, 2024
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Autogen core quickstart guide fails about half the time
4 participants