Skip to content

feat(google_genai): Instrument embed_content methods#111

Open
Abhijeet Prasad (AbhiPrasad) wants to merge 3 commits intomainfrom
abhi-google-genai-embed-instrumentation
Open

feat(google_genai): Instrument embed_content methods#111
Abhijeet Prasad (AbhiPrasad) wants to merge 3 commits intomainfrom
abhi-google-genai-embed-instrumentation

Conversation

@AbhiPrasad
Copy link
Member

resolves #105

This PR adds instrumentation for client.models.embed_content() and client.aio.models.embed_content(). It also adds tests accordingly.

image

While I was working on this, I refactored all the wrapper helpers to be a common set of utilities, which should make this much easier to maintain and work with.

Copy link
Contributor

@viadezo1er ViaDézo1er / cedric (viadezo1er) left a comment

Choose a reason for hiding this comment

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

Assuming the CI passes, good to merge.

Edit: the CI passes!

@viadezo1er
Copy link
Contributor

Though maybe review my commits before merging?

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.

Google GenAI: embed_content not instrumented

2 participants