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

docs: update multi-agent tutorials to use Command #2643

Merged
merged 4 commits into from
Dec 6, 2024

Conversation

vbarda
Copy link
Collaborator

@vbarda vbarda commented Dec 5, 2024

No description provided.

@vbarda vbarda enabled auto-merge (squash) December 6, 2024 15:11
@vbarda vbarda merged commit 5fa80e2 into main Dec 6, 2024
58 checks passed
@vbarda vbarda deleted the vb/update-tutorials-command branch December 6, 2024 15:18
@mathislindner
Copy link
Contributor

Not sure if this is all intended, but I would like to add a comment to make sure this was not a mistake:
There are only conditional edges now (the old mermaid graph used to have uninterrupted edges)

should be:

This is the descriptive representation from the tutorial (where the agent ALWAYS reports back):

image

mermaid graph has changed to:

and here is the new mermaid graph that contradicts the intended behavior: it used to have uninterrupted arrows, but now it only has dotted arrows ( but the agent should always report back to the supervisor , no!?):

image

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.

2 participants