Skip to content

Fix imports and update dependencies #237

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

Merged
merged 5 commits into from
Jun 24, 2025
Merged

Conversation

strickvl
Copy link
Contributor

@strickvl strickvl commented Jun 23, 2025

Summary

  • Add OPENROUTER_API_KEY documentation to README installation instructions
  • Update litellm dependency to >=1.73.0 (remove upper bound constraint)
  • Update langfuse dependency to <3.0.0 for compatibility
  • Fix import issues in tracing_metadata_utils.py

strickvl and others added 3 commits June 23, 2025 17:40
The default configuration uses OpenRouter models which require the
OPENROUTER_API_KEY environment variable to be set.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Use TYPE_CHECKING to avoid runtime imports and fix import issues
with langfuse client types.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Update litellm to >=1.73.0 (remove upper bound)
- Update langfuse to <3.0.0 (compatible version constraint)
- Revert TYPE_CHECKING import pattern in tracing_metadata_utils.py

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link

dagshub bot commented Jun 23, 2025

@strickvl strickvl added bug Something isn't working documentation Improvements or additions to documentation internal labels Jun 23, 2025
@strickvl strickvl requested a review from htahir1 June 23, 2025 15:53
@strickvl strickvl requested a review from htahir1 June 24, 2025 17:03
@htahir1 htahir1 merged commit dc4efbe into main Jun 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation internal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants