Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

set baseURL for text2vec-openai schema config #148

Merged
merged 4 commits into from
May 18, 2024

Conversation

samos123
Copy link
Contributor

@samos123 samos123 commented May 7, 2024

Fixes #147

@samos123 samos123 force-pushed the openai-schema-baseurl branch from 9bef784 to 51f85b9 Compare May 7, 2024 06:23
@samos123
Copy link
Contributor Author

samos123 commented May 7, 2024

edit: ignore, it's working now

@samos123
Copy link
Contributor Author

samos123 commented May 8, 2024

nvm it's working now. Could use a review on this.

@thomashacker
Copy link
Collaborator

Thanks for the PR 🚀 I'll have a look

Copy link
Collaborator

@thomashacker thomashacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks a lot!

@thomashacker thomashacker merged commit 940fbbf into weaviate:main May 18, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: Add baseURL config for Weaviate schema generation
2 participants