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

Add Vertex gen AI response attributes and gen_ai.choice events #3227

Merged
merged 8 commits into from
Feb 7, 2025

Conversation

aabmass
Copy link
Member

@aabmass aabmass commented Feb 3, 2025

Description

Part of #3041

Adds span attributes from the GenAI response. Response events are next PR.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • New test cases

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@aabmass aabmass force-pushed the vertex-response-attrs branch from 82bc592 to 9b581e7 Compare February 3, 2025 17:47
@aabmass aabmass marked this pull request as ready for review February 3, 2025 17:48
@aabmass aabmass requested a review from a team as a code owner February 3, 2025 17:48
@aabmass aabmass force-pushed the vertex-response-attrs branch from 9b581e7 to 7398657 Compare February 3, 2025 23:09
@aabmass aabmass changed the title Add Vertex gen AI response span attributes Add Vertex gen AI response span attributes and gen_ai.choice events Feb 4, 2025
@aabmass aabmass changed the title Add Vertex gen AI response span attributes and gen_ai.choice events Add Vertex gen AI response attributes and gen_ai.choice events Feb 4, 2025
@aabmass aabmass force-pushed the vertex-response-attrs branch from 4a01ed1 to bf60da4 Compare February 4, 2025 22:13
@aabmass aabmass enabled auto-merge (squash) February 7, 2025 22:58
@aabmass aabmass merged commit c0bc2c9 into open-telemetry:main Feb 7, 2025
694 checks passed
@aabmass aabmass deleted the vertex-response-attrs branch February 7, 2025 23:07
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

Successfully merging this pull request may close these issues.

8 participants