-
Notifications
You must be signed in to change notification settings - Fork 798
Add gen_ai_latest_experimental
to the Sem Conv stability flag. Add ContentCapturingMode
enum
#3716
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
Conversation
For some reason |
opentelemetry-instrumentation/src/opentelemetry/instrumentation/_semconv.py
Outdated
Show resolved
Hide resolved
opentelemetry-instrumentation/src/opentelemetry/instrumentation/_semconv.py
Outdated
Show resolved
Hide resolved
what are you getting on your end? Installing ruff==0.6.9 and running it in opentelemetry-util-genai it returns me:
|
…n/_semconv.py Co-authored-by: Emídio Neto <[email protected]>
When I run tox -e ruff (from the root directory if that matters), it doesn't format the files in genai-utils when it clearly should |
util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py
Outdated
Show resolved
Hide resolved
util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py
Outdated
Show resolved
Hide resolved
…ls.py Co-authored-by: Riccardo Magliocchetti <[email protected]>
opentelemetry-instrumentation/src/opentelemetry/instrumentation/_semconv.py
Outdated
Show resolved
Hide resolved
util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py
Outdated
Show resolved
Hide resolved
util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Aaron Abbott <[email protected]>
Co-authored-by: Aaron Abbott <[email protected]>
Co-authored-by: Aaron Abbott <[email protected]>
Co-authored-by: Aaron Abbott <[email protected]>
…python-contrib into gen_ai_flags
util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py
Outdated
Show resolved
Hide resolved
@DylanRussell can you add to the changelog? |
Ack added changelog |
See open-telemetry#3716, the changelog entry should have been in the sub-package specific one.
See open-telemetry#3716, the changelog entry should have been in the sub-package specific one.
Description
Add
gen_ai_latest_experimental
to the Sem Conv stability env var.Add
ContentCapturingMode
enum which will get it's value from theOTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT
env varType of change
Please delete options that are not relevant.
How Has This Been Tested?
Unit tests
Does This PR Require a Core Repo Change?
Checklist:
See contributing.md for styleguide, changelog guidelines, and more.