Remove response.usage.completion_tokens, response.usage.total_tokens, response.usage.prompt_tokens from LlmEmbedding and LlmChatCompletionSummary
#2057
Comments
newrelic-node-agent-team
added this to Triage Needed: Unprioritized Features
in Node.js Engineering Board
kmudduluru
moved this from Triage Needed: Unprioritized Features
to To do: Features here are prioritized
in Node.js Engineering Board
kmudduluru
moved this from To do: Features here are prioritized
to Triage Needed: Unprioritized Features
in Node.js Engineering Board
bizob2828
moved this from Triage Needed: Unprioritized Features
to In progress: Issues being worked on
in Node.js Engineering Board
bizob2828
moved this from In progress: Issues being worked on
to Needs PR Review
in Node.js Engineering Board
Node.js Engineering Board
automation
moved this from Needs PR Review
to Done: Issues recently completed
Apr 1, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Once the AIM UI has been updated to read token counts from
token_countkey(see #2056), we should remove the legacy attributesresponse.usage.completion_tokens,response.usage.total_tokens, andresponse.usage.prompt_tokensfrom openai and langchain. We also want to removetransaction_idfrom all events as it will no longer be necesary as events will be queried by trace_idThe text was updated successfully, but these errors were encountered: