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

chore: Adjust llm.tools semantic convention example #1180

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

nate-mar
Copy link
Contributor

Adjusting the example for llm.tools to make it consistent with how other list of object semantic conventions are repented in the documentation, so that it doesn't convey that there's something actually different in its structures compared to any other list of objects semantic convention, such as llm.input_messages and llm.output_messages. Ultimately, these open inference semantic conventions are flattened and not nested when ingested via OTLP (e.g. llm.tools.0.tool.id, llm.tools.1.tool.id` for semantic conventions that use list of objects), so there is no actual difference in the structure of the data between semantic conventions of the same type.

Furthermore, some systems may choose to represent conventions like a list of objects as a single column as opposed to a flattened set of values, so this doc change also ensures that users know they can expect to consistently handle conventions of the same type (e.g. list of objects).

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Dec 27, 2024
@nate-mar nate-mar merged commit 0b3240b into main Dec 27, 2024
2 checks passed
@nate-mar nate-mar deleted the update-tools-example branch December 27, 2024 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants