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.

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

openai[patch]: Upgrade @root_validators in preparation for pydantic 2 migration #25491

Merged
merged 16 commits into from
Sep 3, 2024
Merged

openai[patch]: Upgrade @root_validators in preparation for pydantic 2 migration #25491

merged 16 commits into from
Sep 3, 2024

Conversation

eyurtsev
Copy link
Collaborator

@eyurtsev eyurtsev commented Aug 16, 2024 •
edited
Loading

  • Upgrade @root_validator in openai pkg
  • Ran notebooks for all but AzureAI embeddings

@vercel Vercel
Copy link

vercel bot commented Aug 16, 2024 •
edited
Loading

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Sep 3, 2024 9:39pm

@efriis efriis added the partner label Aug 16, 2024
@efriis efriis self-assigned this Aug 16, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 🔌: openai Primarily related to OpenAI integrations 🤖:improvement Medium size change to existing code to handle new use-cases labels Aug 16, 2024
@eyurtsev eyurtsev added the 0.3 prep Work done for 0.3 prep label Aug 16, 2024
@eyurtsev eyurtsev mentioned this pull request Sep 3, 2024
Merged
eyurtsev added a commit that referenced this pull request Sep 3, 2024
@eyurtsev
core[minor]: Add support for multiple env keys for secrets_from_env (#…
fa8402e 
…25971)

- Add support to look up secret using more than one env variable
- Add overload to help mypy

Needed for #25491
@baskaryan baskaryan merged commit bc3b851 into master Sep 3, 2024
24 checks passed
@baskaryan baskaryan deleted the eugene/openai_root_validators branch September 3, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@efriis efriis

Labels
0.3 prep Work done for 0.3 prep 🤖:improvement Medium size change to existing code to handle new use-cases 🔌: openai Primarily related to OpenAI integrations partner size:L This PR changes 100-499 lines, ignoring generated files.
Projects
PR Reviews
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eyurtsev @efriis @baskaryan