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

Magentic one example #3802

Merged
merged 11 commits into from
Oct 17, 2024
Merged

Conversation

husseinmozannar
Copy link
Contributor

Why are these changes needed?

Magentic-One has no example script showcasing how to create the agent team to solve tasks.

I create an example.py script to showcase magentic-one.

I also edited the orchestrator to return a final answer and not just the ledger.

I added minor comments to various magentic-one components to improve readability.

Related issue number

Checks

@husseinmozannar husseinmozannar marked this pull request as ready for review October 16, 2024 05:25
@husseinmozannar
Copy link
Contributor Author

I'm still new to uv, not sure why uv.lock changed.

@afourney @gagb I added a simple example script for magentic-one, let's keep brainstorming about the API for magentic-one and migration to agent-chat

@husseinmozannar husseinmozannar marked this pull request as draft October 16, 2024 17:39
@husseinmozannar
Copy link
Contributor Author

I think it will be valuable to create a frontend interface that visualizes the agent graph + ledger + chat, to create demos for magentic-one release - keeping this PR a draft while I work on that

Copy link
Member

@afourney afourney left a comment

Choose a reason for hiding this comment

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

Looks good.

@rysweet
Copy link
Collaborator

rysweet commented Oct 17, 2024

@husseinmozannar - is this still meant to be in draft?

@husseinmozannar husseinmozannar marked this pull request as ready for review October 17, 2024 15:32
@husseinmozannar
Copy link
Contributor Author

husseinmozannar commented Oct 17, 2024

@rysweet

@husseinmozannar - is this still meant to be in draft?

Just added a readme for the examples.

Feel free to merge now, instead of the interface, we will start with a log viewer. Thank you.

@rysweet rysweet merged commit 50cd4cc into microsoft:main Oct 17, 2024
30 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.

3 participants