Skip to content

feat(openai): Populate OpenAI service tier token details #32721

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

feat(openai): Populate OpenAI service tier token details #32721

wants to merge 6 commits into from
+112 −20

Conversation

jacoblee93
Copy link
Contributor

@vercel Vercel
Copy link

vercel bot commented Aug 27, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
langchain Ignored Ignored Preview Aug 27, 2025 7:34pm

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 27, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #32721 will not alter performance

Comparing jacob/service_tier (ab745f9) with master (1cf851e)1

Summary

✅ 6 untouched
⏩ 28 skipped2

Footnotes

  1. No successful run was found on master (637777b) during the generation of this report, so 1cf851e was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

  2. 28 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 27, 2025 •
edited
Loading

CodSpeed Instrumentation Performance Report

Merging #32721 will not alter performance

Comparing jacob/service_tier (594d2f6) with master (7f9b077)

Summary

✅ 14 untouched benchmarks

@mdrxy mdrxy added the integration Related to a provider partner package integration label Aug 28, 2025
@baskaryan baskaryan requested review from ccurme and mdrxy September 2, 2025 18:24
@jacoblee93 jacoblee93 marked this pull request as ready for review September 2, 2025 19:08
@jacoblee93
Copy link
Contributor Author

Bump CC @ccurme!

ccurme added 3 commits October 13, 2025 15:11
@ccurme
Merge branch 'master' into jacob/service_tier
eeb27d3 
# Conflicts:
#	libs/partners/openai/langchain_openai/chat_models/base.py
#	libs/partners/openai/tests/integration_tests/chat_models/test_base.py
@ccurme
lint
29cfc55
@ccurme
add test for responses API
ab745f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ccurme ccurme Awaiting requested review from ccurme ccurme is a code owner

@mdrxy mdrxy Awaiting requested review from mdrxy mdrxy is a code owner

Assignees

@ccurme ccurme

Labels

integration Related to a provider partner package integration

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jacoblee93 @ccurme @mdrxy