-
Notifications
You must be signed in to change notification settings - Fork 81
Specify aiohttp version to avoid import errors, update Langfuse to v3 SDK #74
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
base: main
Are you sure you want to change the base?
Specify aiohttp version to avoid import errors, update Langfuse to v3 SDK #74
Conversation
Could you please confirm if langfuse.fetch_observations and langfuse.score also need an update in v3 migration and the behaviour is consistent with Langfuse 2.60.8 ? |
Updated with latest commit - please merge |
Please move deploy command cell to after installing requirements.txt |
Corrected
wt., 24 cze 2025 o 16:08 mvangara10 ***@***.***> napisał(a):
… *mvangara10* left a comment (strands-agents/samples#74)
<#74 (comment)>
Please move deploy command cell to after installing requirements.txt
—
Reply to this email directly, view it on GitHub
<#74 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANGCEZBH7RQ7ZFF54HDUWWD3FFLUPAVCNFSM6AAAAAB7P7A4HOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTAMBQGY3DCNBTGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Issue #, if available:
Description of changes:
Older versions of aiohttp will raise error during litellm import in this notebook
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.