-
Notifications
You must be signed in to change notification settings - Fork 516
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
[opentelemetry-instrumentation-openai] Add openai sdk #2378
Comments
@pichlermarc Does this instrumentation request good to get started ? |
@AmanAgarwal041 thanks for opening the request A few more questions:
The reason I'm asking is that we're overwhelmed here with different instrumentation packages as we have to be familiar with the source code of the package we're instrumenting - we've had numerous component owners jump ship after contributing a component and it's simply not feasible for maintainers of this repo to be experts in the inner workings of 40+ wildly different packages. As this is the case you'd still be mostly maintaining this yourself here, as you would in a separate repo - but with less permissions and less control over what you can do and cannot do. |
@pichlermarc to answer your questions
I understand the maintaining it in a separate repo is lesser pain but keeping it here would actually make it trust worthy and genuine to get adopted. |
@pichlermarc I have created a draft PR for now. |
Is your instrumentation request related to a problem? Please describe
Add an instrument for openai to handle telemetry for chat completions
Is it applicable for Node or Browser or both?
Its applicable for Node
Do you expect this instrumentation to be commonly used?
Yes
Weekly Downloads:
What version of instrumentation are you interested in using?
Versions:
Code Owners
@AmanAgarwal041
@patcher9
Additional context
The text was updated successfully, but these errors were encountered: