-
Notifications
You must be signed in to change notification settings - Fork 70
[LCORE-873] Support Solr Vector I/O Provider in LCORE #868
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?
Changes from all commits
1ea5f90
2428b48
19bc8ca
fdd4028
31ffb01
c5cfefb
d834cff
cb1c532
abccf07
c7e74f5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,3 +30,7 @@ conversation_cache: | |
|
|
||
| authentication: | ||
| module: "noop" | ||
|
|
||
|
|
||
| solr: | ||
| offline: True | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,11 @@ | |
| from utils.common import register_mcp_servers_async | ||
| from utils.llama_stack_version import check_llama_stack_version | ||
|
|
||
| import faulthandler | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Probably leftover too |
||
| import signal | ||
|
|
||
| faulthandler.register(signal.SIGUSR1) | ||
|
|
||
| logger = get_logger(__name__) | ||
|
|
||
| logger.info("Initializing app") | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -127,7 +127,7 @@ | |
| MCP_AUTH_CLIENT = "client" | ||
|
|
||
| # default RAG tool value | ||
| DEFAULT_RAG_TOOL = "knowledge_search" | ||
| DEFAULT_RAG_TOOL = "file_search" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This causes error in integration tests, please rename it also there.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what do you mean rename?
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you look at the integration tests job in the original branch you see something like: That means you renamed the constant in code but not in integration tests.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ohh got it. Thank you! |
||
|
|
||
| # Media type constants for streaming responses | ||
| MEDIA_TYPE_JSON = "application/json" | ||
|
|
@@ -168,3 +168,11 @@ | |
| # quota limiters constants | ||
| USER_QUOTA_LIMITER = "user_limiter" | ||
| CLUSTER_QUOTA_LIMITER = "cluster_limiter" | ||
|
|
||
| # Vector search constants | ||
| VECTOR_SEARCH_DEFAULT_K = 5 | ||
| VECTOR_SEARCH_DEFAULT_SCORE_THRESHOLD = 0.0 | ||
| VECTOR_SEARCH_DEFAULT_MODE = "hybrid" | ||
|
|
||
| # SOLR OKP RAG | ||
| MIMIR_DOC_URL = "https://mimir.corp.redhat.com" | ||
Uh oh!
There was an error while loading. Please reload this page.