Skip to content

[codex-analytics] add token usage metadata#16641

Draft
rhan-oai wants to merge 1 commit intopr16639from
pr16640
Draft

[codex-analytics] add token usage metadata#16641
rhan-oai wants to merge 1 commit intopr16639from
pr16640

Conversation

@rhan-oai
Copy link
Copy Markdown
Collaborator

@rhan-oai rhan-oai commented Apr 3, 2026

@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 18:54
@rhan-oai rhan-oai force-pushed the pr16640 branch 2 times, most recently from 6e02a3f to d3f41f1 Compare April 7, 2026 19:29
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 19:32
@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 19:37
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 19:40
@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 20:42
@rhan-oai rhan-oai force-pushed the pr16640 branch 3 times, most recently from 0bb2e8d to df708ee Compare April 7, 2026 21:11
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 21:15
@rhan-oai rhan-oai requested a review from owenlin0 April 7, 2026 21:18
turn_id,
token_usage,
};
if let Some(analytics_events_client) = analytics_events_client {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still need these changes now that we're emitting a custom turn token fact?

@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 21:53
@rhan-oai rhan-oai force-pushed the pr16640 branch 2 times, most recently from 2397bff to df708ee Compare April 7, 2026 22:08
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 22:09
@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 22:14
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 22:14
@rhan-oai rhan-oai changed the base branch from pr16639 to main April 7, 2026 22:53
@rhan-oai rhan-oai changed the base branch from main to pr16639 April 7, 2026 23:37
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.

2 participants