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

Zep sdk version #5267

Merged
merged 11 commits into from
May 25, 2023
Merged

Zep sdk version #5267

merged 11 commits into from
May 25, 2023

Conversation

dev2049
Copy link
Contributor

@dev2049 dev2049 commented May 25, 2023

zep-python's sync methods no longer need an asyncio wrapper. This was causing issues with FastAPI deployment.
Zep also now supports putting and getting of arbitrary message metadata.

Bump zep-python version to v0.30

Remove nest-asyncio from Zep example notebooks.

Modify tests to include metadata.

@dev2049
Copy link
Contributor Author

dev2049 commented May 25, 2023

changed so only zep is updated in poetry.lock (poetry lock --no-update instead of poetry lock)

@dev2049 dev2049 mentioned this pull request May 25, 2023
@dev2049 dev2049 added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label May 25, 2023
@dev2049 dev2049 merged commit ca88b25 into master May 25, 2023
@dev2049 dev2049 deleted the zep-sdk-version branch May 25, 2023 20:42
@danielchalef danielchalef mentioned this pull request Jun 5, 2023
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
zep-python's sync methods no longer need an asyncio wrapper. This was
causing issues with FastAPI deployment.
Zep also now supports putting and getting of arbitrary message metadata.

Bump zep-python version to v0.30

Remove nest-asyncio from Zep example notebooks.

Modify tests to include metadata.

---------

Co-authored-by: Daniel Chalef <daniel.chalef@private.org>
Co-authored-by: Daniel Chalef <131175+danielchalef@users.noreply.github.com>
This was referenced Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants