Skip to content

feat(fireworks,groq,openrouter): add standard model property#35542

Merged
Mason Daugherty (mdrxy) merged 1 commit intomasterfrom
mdrxy/std-model-prop
Mar 3, 2026
Merged

feat(fireworks,groq,openrouter): add standard model property#35542
Mason Daugherty (mdrxy) merged 1 commit intomasterfrom
mdrxy/std-model-prop

Conversation

@mdrxy
Copy link
Copy Markdown
Member

Add a model property to ChatFireworks, ChatGroq, and ChatOpenRouter that returns model_name. These partners use Pydantic's Field(alias="model") on model_name, which means instance.model doesn't work as a read accessor after construction — it raises an AttributeError or returns the field descriptor. ChatOpenAI already has this property; this brings the remaining in-repo partners to parity.

@github-actions github-actions bot added integration PR made that is related to a provider partner package integration fireworks `langchain-fireworks` package issues & PRs groq `langchain-groq` package issues & PRs openrouter `langchain-openrouter` package issues & PRs feature For PRs that implement a new feature; NOT A FEATURE REQUEST internal labels Mar 3, 2026
@mdrxy Mason Daugherty (mdrxy) merged commit e50625e into master Mar 3, 2026
72 checks passed
@mdrxy Mason Daugherty (mdrxy) deleted the mdrxy/std-model-prop branch March 3, 2026 22:28
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 3, 2026

Merging this PR will not alter performance

✅ 3 untouched benchmarks
⏩ 33 skipped benchmarks1


Comparing mdrxy/std-model-prop (294b659) with master (6b37ad4)

Open in CodSpeed

Footnotes

  1. 33 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature For PRs that implement a new feature; NOT A FEATURE REQUEST fireworks `langchain-fireworks` package issues & PRs groq `langchain-groq` package issues & PRs integration PR made that is related to a provider partner package integration internal openrouter `langchain-openrouter` package issues & PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant