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

Look into the naming of workflow_context_properties #25

Open
kshitiz-okahu opened this issue Aug 8, 2024 · 3 comments
Open

Look into the naming of workflow_context_properties #25

kshitiz-okahu opened this issue Aug 8, 2024 · 3 comments

Comments

@kshitiz-okahu
Copy link
Contributor

The proposal is to decide on naming of attribute for propagation of workflow context across GenAI traces.

@kshitiz-okahu
Copy link
Contributor Author

kshitiz-okahu commented Aug 20, 2024

after looking at the OpenTelemetry standards for Gen AI, something like the following should work
gen_ai.context_propogation

Context propagation is a concept(https://opentelemetry.io/docs/concepts/context-propagation/) in OpenTelemetry so the part context_propogation is added.
The prefix gen_ai is the experimental prefix today in OpenTelemetry used for Gen AI request and responses.

@kshitiz-okahu
Copy link
Contributor Author

The consensus is that the naming should not include any genAI references.

I am in favour of naming these fields as follows

context.session_id

@oi-raanne
Copy link
Contributor

'invocation' would be better than 'context', as 'context' is first class object in open telemetry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants