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

Add Zep ecosystem doc and notebook #3681

Merged
merged 10 commits into from
Oct 17, 2024

Conversation

danielchalef
Copy link

Why are these changes needed?

Added: Ecosystem markdown and Jupyter notebook demonstrating how to build Autogen agents with Zep's memory layer.

Related issue number

Checks

@danielchalef
Copy link
Author

@microsoft-github-policy-service agree company="Zep Software, Inc."

@rysweet rysweet requested a review from jackgerrits October 10, 2024 20:20
@rysweet
Copy link
Collaborator

rysweet commented Oct 10, 2024

Hi @danielchalef - with the new release of autogen 0.4 we are going to be shortly coming up with a policy to provide clarity on the best way to create, discover, and share 3rd party integrations with AutoGen. Stay Tuned

@rysweet rysweet added 0.2 Issues which are related to the pre 0.4 codebase awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster labels Oct 10, 2024
@rysweet
Copy link
Collaborator

rysweet commented Oct 10, 2024

This PR is against AutoGen 0.2. AutoGen 0.2 has been moved to the 0.2 branch. Please rebase your PR on the 0.2 branch or update it to work with the new AutoGen 0.4 that is now in main.

@danielchalef danielchalef changed the base branch from main to 0.2 October 10, 2024 21:54
@danielchalef
Copy link
Author

@rysweet Rebased on 0.2. Base branch for PR switched to 0.2.

@danielchalef
Copy link
Author

@jackgerrits @rysweet fixed formatting and spelling. pre-commit tests now pass.

@rysweet rysweet removed the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 11, 2024
@rysweet
Copy link
Collaborator

rysweet commented Oct 11, 2024

Thanks @danielchalef!

@jackgerrits I'm not sure where this one falls wrt the proposed policy updates.

@jackgerrits
Copy link
Member

We should take the 0.2 one, and then for 0.4 would you check this and consider following the discovery guidance? https://microsoft.github.io/autogen/dev/user-guide/extensions-user-guide/index.html#creating-your-own-community-extension @danielchalef

@danielchalef
Copy link
Author

We should take the 0.2 one, and then for 0.4 would you check this and consider following the discovery guidance? https://microsoft.github.io/autogen/dev/user-guide/extensions-user-guide/index.html#creating-your-own-community-extension @danielchalef

Yeah - will be happy to do so.

@rysweet rysweet added the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 16, 2024
@jackgerrits
Copy link
Member

Just realized why the CI is failing for this one @danielchalef. The notebook needs to be edited so that is displayed on the website (just adding some metadata). See here: https://github.com/microsoft/autogen/blob/0.2/notebook/contributing.md

Once that is done the link checker should be happy

@danielchalef
Copy link
Author

@jackgerrits Oops. I missed that. Fixed. Thx!

Copy link
Collaborator

@rysweet rysweet left a comment

Choose a reason for hiding this comment

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

thanks!

@rysweet rysweet merged commit 76a4bd0 into microsoft:0.2 Oct 17, 2024
18 checks passed
@danielchalef
Copy link
Author

@rysweet thanks!

@danielchalef danielchalef deleted the feat/zep-notebook branch October 17, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.2 Issues which are related to the pre 0.4 codebase awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants