Skip to content
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.

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

premai[patch]:Standardized model init args #21308

Merged
merged 2 commits into from
May 6, 2024
Merged

Conversation

Narapady
Copy link
Contributor

@Narapady Narapady commented May 5, 2024

Standardized model init args #20085

  • Enable premai chat model to be initialized with model_name as an alias for model, api_key as an alias for premai_api_key.
  • Add initialization test test_premai_initialization

Copy link

vercel bot commented May 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview May 6, 2024 10:08pm

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. 🤖:improvement Medium size change to existing code to handle new use-cases labels May 5, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label May 6, 2024
@hwchase17 hwchase17 self-assigned this May 6, 2024
@ccurme ccurme merged commit 95cc8e3 into langchain-ai:master May 6, 2024
59 checks passed
dglogo pushed a commit to dglogo/langchain that referenced this pull request May 8, 2024
[Standardized model init args
langchain-ai#20085](langchain-ai#20085)
- Enable premai chat model to be initialized with `model_name` as an
alias for `model`, `api_key` as an alias for `premai_api_key`.
- Add initialization test `test_premai_initialization`

---------

Co-authored-by: Chester Curme <[email protected]>
kyle-cassidy pushed a commit to kyle-cassidy/langchain that referenced this pull request May 10, 2024
[Standardized model init args
langchain-ai#20085](langchain-ai#20085)
- Enable premai chat model to be initialized with `model_name` as an
alias for `model`, `api_key` as an alias for `premai_api_key`.
- Add initialization test `test_premai_initialization`

---------

Co-authored-by: Chester Curme <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants