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

Add new model provider Novita AI #360

Merged
merged 6 commits into from
Jan 27, 2025
Merged

Conversation

jasonhp
Copy link
Contributor

@jasonhp jasonhp commented Jan 6, 2025

Add new model provider Novita AI

Copy link

@orca-security-eu orca-security-eu bot left a 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
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 1   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca
🛡️ The following SAST misconfigurations have been detected
NAME FILE
medium Missing timeout in requests module can cause DoS ...n/NovitaGenerator.py View in code

@weaviate-git-bot
Copy link

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.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

@thomashacker
Copy link
Collaborator

Thanks a lot for your PR! We'll look into it 🔥

@jasonhp
Copy link
Contributor Author

jasonhp commented Jan 10, 2025

Thanks a lot for your PR! We'll look into it 🔥

@thomashacker Hello! I’m wondering when this PR might be reviewed and approved. Thank you!

@thomashacker
Copy link
Collaborator

Hey @jasonhp , we'll try to approve and merge PR as soon as possible 🚀

@vercelInfrAI
Copy link
Contributor

@thomashacker Can you help to approve and merge? Thanks a lot

@jasonhp
Copy link
Contributor Author

jasonhp commented Jan 21, 2025

@thomashacker Hi! I believe it’s ready for review and would be great to have it merged soon if everything looks good to you. Please let me know if there’s anything I should adjust.

@thomashacker
Copy link
Collaborator

Will do next week! Super sorry for the delay and inconvenience 🙏

@thomashacker thomashacker changed the base branch from main to 2.1.2 January 27, 2025 09:35
@thomashacker
Copy link
Collaborator

Thanks a lot for the great work! Will release 2.1.2 today!

@thomashacker thomashacker merged commit fa6fbe1 into weaviate:2.1.2 Jan 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants