[Fix]: Incompatibilities with OpenTelemetry LLM semantics pending release #347
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview:
This PR introduces changes that makes sure OpenLIT is following the OTel Semantic conventions for GenAI
resolves #342
Changes:
Changes in the SDK
gen_ai.request.top_p
is made a doublegen_ai.request.max_tokens
as-1
instead of a stringgen_ai.request.temperature
is made a doublegen_ai.request.presence_penalty
is made a doublegen_ai.request.frequency_penalty
is made a doublegen_ai.response.finish_reason'
is renamed to 'gen_ai.response.finish_reasons'gen_ai.usage.completion_tokens
is renamed togen_ai.usage.output_tokens
gen_ai.usage.prompt_tokens
is renamed togen_ai.usage.input_tokens
Changes in the UI
gen_ai.response.finish_reason'
is renamed to 'gen_ai.response.finish_reasons'gen_ai.usage.completion_tokens
is renamed togen_ai.usage.output_tokens
gen_ai.usage.prompt_tokens
is renamed togen_ai.usage.input_tokens
Visuals (If applicable):
NA
Checklist:
[Feat]: ...
or[Fix]: ....