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

Update langchain handler for v1 #109

Merged
merged 12 commits into from
Mar 21, 2024
Merged

Update langchain handler for v1 #109

merged 12 commits into from
Mar 21, 2024

Conversation

HowieG
Copy link
Contributor

@HowieG HowieG commented Mar 20, 2024

📥 Pull Request

📘 Description
Ported to new event types

🔄 Related Issue (if applicable)
Linear

🎯 Goal
Implementing langchain callbacks

🔍 Additional Context
Async still needs three callbacks implemented
Lingering TODOs

🧪 Testing
Run tests/langchain_handlers/test_langchain_handler.py

Thank you for your contribution to Agentops!

@HowieG HowieG requested a review from bboynton97 March 20, 2024 17:55
Copy link
Contributor

@siyangqiu siyangqiu 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 go!

agentops/event.py Outdated Show resolved Hide resolved
agentops/langchain_callback_handler.py Outdated Show resolved Hide resolved
agentops/langchain_callback_handler.py Outdated Show resolved Hide resolved
agentops/enums.py Outdated Show resolved Hide resolved
@bboynton97 bboynton97 requested a review from siyangqiu March 21, 2024 22:11
@bboynton97 bboynton97 merged commit 91df7bf into main Mar 21, 2024
1 check passed
@siyangqiu siyangqiu deleted the eng-101-langchain-handler branch March 21, 2024 22:55
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