Skip to content

feat(openai): enable stream_usage when using default base URL and client#33296

Merged
ccurme (ccurme) merged 3 commits intov0.3from
cc/stream_usage
Oct 6, 2025
Merged

feat(openai): enable stream_usage when using default base URL and client#33296
ccurme (ccurme) merged 3 commits intov0.3from
cc/stream_usage

Conversation

@ccurme
Copy link
Copy Markdown
Collaborator

@ccurme ccurme (ccurme) commented Oct 6, 2025 •
edited
Loading

Cherry-pick of #33205 and #33298

@ccurme
feat(openai): enable stream_usage when using default base URL and cli…
f780569 
…ent (#33205)

# Conflicts:
#	libs/partners/openai/langchain_openai/chat_models/base.py
@ccurme
fix api ref syntax
fe77ed7
@github-actions github-actions bot added feature For PRs that implement a new feature; NOT A FEATURE REQUEST integration PR made that is related to a provider partner package integration langchain `langchain` package issues & PRs v1 Issue specific to LangChain 1.0 labels Oct 6, 2025
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Oct 6, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #33296 will not alter performance

Comparing cc/stream_usage (c128277) with master (95a451e)1

Summary

✅ 6 untouched
⏩ 28 skipped2

Footnotes

  1. No successful run was found on v0.3 (0b51de4) during the generation of this report, so master (95a451e) 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. ↩

@ccurme
fix(openai): disable stream_usage in chat completions if OPENAI_BASE_…
c128277 
…URL is set (#33298)

This env var is used internally by the OpenAI client.
@ccurme ccurme (ccurme) merged commit 3a465d6 into v0.3 Oct 6, 2025
74 checks passed
@ccurme ccurme (ccurme) deleted the cc/stream_usage branch October 6, 2025 14:22
@eyurtsev Eugene Yurtsev (eyurtsev) mentioned this pull request Nov 19, 2025
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature For PRs that implement a new feature; NOT A FEATURE REQUEST integration PR made that is related to a provider partner package integration langchain `langchain` package issues & PRs v1 Issue specific to LangChain 1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ccurme