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

[OPIK-680] [FR]: Cost Tracking for VertexAI and LangChain #1055

Merged
merged 17 commits into from
Jan 23, 2025
Merged

Conversation

japdubengsub
Copy link
Contributor

@japdubengsub japdubengsub commented Jan 15, 2025

Details

Token usage info is now converted to OpenAI format and sent when using Vertex AI and LangChain

Resolves #
#967

@japdubengsub japdubengsub changed the title OPIK-680 [OPIK-680] [FR]: Cost Tracking for VertexAI and LangChain Jan 15, 2025
Copy link
Collaborator

@alexkuzmik alexkuzmik left a comment

Choose a reason for hiding this comment

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

It would be nice to add 1 integration test which will check usage collection from google provider.

sdks/python/src/opik/integrations/helpers.py Outdated Show resolved Hide resolved
@alexkuzmik
Copy link
Collaborator

@japdubengsub I approved but created a ticket to address the tech debt with a vertex if-statement inside Opik.span.

@japdubengsub japdubengsub marked this pull request as ready for review January 23, 2025 11:54
@japdubengsub japdubengsub requested review from a team as code owners January 23, 2025 11:54
@alexkuzmik alexkuzmik merged commit 9418491 into main Jan 23, 2025
40 checks passed
@alexkuzmik alexkuzmik deleted the OPIK-680 branch January 23, 2025 13:21
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