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

[DERCBOT-1138] PGVector #1735

Merged

Conversation

assouktim
Copy link
Contributor

@assouktim assouktim commented Sep 4, 2024

Here are the changes to environment variables :

Gen AI Orchestrator Server :

Replaced :
tock_gen_ai_orchestrator_open_search_host -> tock_gen_ai_orchestrator_vector_store_host
tock_gen_ai_orchestrator_open_search_port -> tock_gen_ai_orchestrator_vector_store_port
tock_gen_ai_orchestrator_open_search_user -> tock_gen_ai_orchestrator_vector_store_user
tock_gen_ai_orchestrator_open_search_pwd -> tock_gen_ai_orchestrator_vector_store_pwd

Added :
tock_gen_ai_orchestrator_vector_store_provider
tock_gen_ai_orchestrator_vector_store_database
tock_gen_ai_orchestrator_vector_store_test_query
tock_gen_ai_orchestrator_vector_store_secret_manager_provider
tock_gen_ai_orchestrator_vector_store_credentials_secret_name

Deleted :
tock_gen_ai_orchestrator_open_search_timeout
tock_gen_ai_orchestrator_open_search_aws_secret_manager_name

Bot Admin :

Added :
tock_gen_ai_orchestrator_vector_store_provider

Bot Api :

Added :
tock_gen_ai_orchestrator_vector_store_provider

@assouktim assouktim changed the title [DERCBOT-1036] Vector Store Providers [DERCBOT-1038] PGVector Sep 4, 2024
@assouktim assouktim force-pushed the feature/dercbot-1138 branch 3 times, most recently from efb2f47 to 1491c9c Compare September 10, 2024 12:46
@assouktim assouktim force-pushed the feature/dercbot-1138 branch 2 times, most recently from 94ab536 to 6aaa152 Compare September 24, 2024 14:12
@assouktim assouktim marked this pull request as ready for review September 26, 2024 07:36
Copy link
Contributor

@killian-mahe killian-mahe left a comment

Choose a reason for hiding this comment

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

Didn't review the Kotlin part, but thet PR is good on the server part.

@assouktim assouktim force-pushed the feature/dercbot-1138 branch 2 times, most recently from 88ba6a6 to e532c83 Compare October 8, 2024 14:02
Copy link
Member

@Benvii Benvii left a comment

Choose a reason for hiding this comment

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

Good for me thanks a lot for the updates good job, some issue created for futur improvements.

@Benvii Benvii changed the title [DERCBOT-1038] PGVector [DERCBOT-1138] PGVector Oct 9, 2024
@Benvii Benvii merged commit c9f252f into theopenconversationkit:master Oct 9, 2024
@Benvii Benvii deleted the feature/dercbot-1138 branch October 9, 2024 07:48
@vsct-jburet vsct-jburet added this to the 24.9.2 milestone Oct 9, 2024
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.

5 participants