Skip to content

Conversation

@AWarno
Copy link
Contributor

@AWarno AWarno commented Sep 23, 2025

  1. Add total stats.
  2. Add reasoning token stats (if provided). - https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens" in usage, (completion_tokens_details, output_tokens_details)
  3. Make stats cache-resistant — do not include stats if the response is from cache.

@AWarno AWarno requested review from a team as code owners September 23, 2025 08:54
@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 23, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@AWarno AWarno force-pushed the awarno/reasoning-tokens branch from 8c5c7c2 to 4d79657 Compare September 23, 2025 08:55
@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test 8c5c7c2

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 23, 2025

/ok to test 8c5c7c2

@AWarno, there was an error processing your request: E2

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/2/

@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test 4d79657

@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test a1f8ee0

@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test 484ee01

@AWarno AWarno force-pushed the awarno/reasoning-tokens branch from a748a71 to bcb8ce0 Compare September 23, 2025 11:08
@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test bcb8ce0

@AWarno
Copy link
Contributor Author

AWarno commented Sep 23, 2025

/ok to test f28afb4

@AWarno AWarno merged commit 9367ef0 into main Sep 25, 2025
34 checks passed
@AWarno AWarno deleted the awarno/reasoning-tokens branch September 25, 2025 08:06
AWarno added a commit that referenced this pull request Sep 25, 2025
1. Add total stats.
2. Add reasoning token stats (if provided). -
https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens"
in usage, (completion_tokens_details, output_tokens_details)
3. Make stats cache-resistant — do not include stats if the response is
from cache.

---------

Signed-off-by: Anna Warno <[email protected]>
AWarno added a commit that referenced this pull request Sep 25, 2025
1. Add total stats.
2. Add reasoning token stats (if provided). -
https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens"
in usage, (completion_tokens_details, output_tokens_details)
3. Make stats cache-resistant — do not include stats if the response is
from cache.

---------

Signed-off-by: Anna Warno <[email protected]>
AWarno added a commit that referenced this pull request Sep 26, 2025
1. Add total stats.
2. Add reasoning token stats (if provided). -
https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens"
in usage, (completion_tokens_details, output_tokens_details)
3. Make stats cache-resistant — do not include stats if the response is
from cache.

---------

Signed-off-by: Anna Warno <[email protected]>
AWarno added a commit that referenced this pull request Sep 29, 2025
1. Add total stats.
2. Add reasoning token stats (if provided). -
https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens"
in usage, (completion_tokens_details, output_tokens_details)
3. Make stats cache-resistant — do not include stats if the response is
from cache.

---------

Signed-off-by: Anna Warno <[email protected]>
AWarno added a commit that referenced this pull request Oct 23, 2025
1. Add total stats.
2. Add reasoning token stats (if provided). -
https://platform.openai.com/docs/guides/reasoning or "reasoning_tokens"
in usage, (completion_tokens_details, output_tokens_details)
3. Make stats cache-resistant — do not include stats if the response is
from cache.

---------

Signed-off-by: Anna Warno <[email protected]>
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.

3 participants