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

Move GenAI common utils into opentelemetry-instrumentation package #3191

Open
aabmass opened this issue Jan 15, 2025 · 1 comment
Open

Move GenAI common utils into opentelemetry-instrumentation package #3191

aabmass opened this issue Jan 15, 2025 · 1 comment
Labels
enhancement New feature or request gen-ai Related to generative AI

Comments

@aabmass
Copy link
Member

aabmass commented Jan 15, 2025

So that they can be used across those packages. OpenAI, Cohere, and Vertex instrumentations need some common utils. See https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3081/files/#r1884751133

They will be available in the opentelemetry.instrumentation.gen_ai module.

Because GenAI packages are not released in lockstep with the rest of the repo, we will need to first release this change before updating those instrumentation packages to use it. See this comment #3188 (comment)

@aabmass
Copy link
Member Author

aabmass commented Jan 16, 2025

Discussed in Python SIG. Given there are only a few utils, I think we might table this PR until we have more need for code sharing.

It would be better to put it into a new shared package opentelemetry-util-gen-ai which is also released independently. That would help us move faster.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gen-ai Related to generative AI
Development

No branches or pull requests

1 participant