Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

Bedrock: Remove response.usage.completion_tokens, response.usage.total_tokens, response.usage.prompt_tokens from LlmEmbedding and LlmChatCompletionSummary #265

Closed
bizob2828 opened this issue Feb 29, 2024 · 2 comments · Fixed by #278
Closed

Bedrock: Remove response.usage.completion_tokens, response.usage.total_tokens, response.usage.prompt_tokens from LlmEmbedding and LlmChatCompletionSummary #265

bizob2828 opened this issue Feb 29, 2024 · 2 comments · Fixed by #278
Assignees
@bizob2828
Labels
points:3 A few days

Comments

@bizob2828
Copy link
Member

bizob2828 commented Feb 29, 2024 •
edited
Loading

Description

Once the AIM UI has been updated to read token counts from token_count key(see #264 ), we should remove the legacy attributes response.usage.completion_tokens, response.usage.total_tokens, and response.usage.prompt_tokens.
We also want to remove transaction_id from all events as it will no longer be necesary as events will be queried by trace_id
@workato-integration
Copy link

https://new-relic.atlassian.net/browse/NR-234564

@bizob2828 bizob2828 added the points:3 A few days label Mar 1, 2024
@jsumners-nr
Copy link
Contributor

See #269

@bizob2828 bizob2828 mentioned this issue Mar 11, 2024
Merged
@bizob2828 bizob2828 self-assigned this Apr 1, 2024
@bizob2828 bizob2828 mentioned this issue Apr 1, 2024
Merged
@bizob2828 bizob2828 moved this to Done: Issues recently completed in Node.js Engineering Board Jul 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bizob2828 bizob2828

Labels
points:3 A few days
Projects
Node.js Engineering Board
Archived in project
Milestone
No milestone
2 participants
@bizob2828 @jsumners-nr