-
Notifications
You must be signed in to change notification settings - Fork 100
chore: #1763 #1776
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
base: main
Are you sure you want to change the base?
chore: #1763 #1776
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Orca Security Scan Summary
Status | Check | Issues by priority | |
---|---|---|---|
![]() |
Infrastructure as Code | ![]() ![]() ![]() ![]() |
View in Orca |
![]() |
SAST | ![]() ![]() ![]() ![]() |
View in Orca |
![]() |
Secrets | ![]() ![]() ![]() ![]() |
View in Orca |
![]() |
Vulnerabilities | ![]() ![]() ![]() ![]() |
View in Orca |
I agree with the CLA |
), | ||
( | ||
Configure.Vectorizer.multi2vec_palm( | ||
Configure.Vectorizer.multi2vec_palm( # not changed to _google to ensure BC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kept these to ensure deprecated methods are tested until they're removed
), | ||
( | ||
Configure.Vectorizer.text2vec_palm( | ||
Configure.Vectorizer.text2vec_palm( # not changed to text2vec_google to ensure BC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kept these to ensure deprecated methods are tested until they're removed
Updated:
Let me know if I missed anything. |
To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge. |
Thank you for your contribution - we still have a few customers on 1.27 images. I'll merge this once we upgraded everyone |
@dirkkul - sounds great. Thanks for the hard work! |
Closes #1763