-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: document search sources string resolver #264
feat: document search sources string resolver #264
Conversation
Code Coverage Summary
Diff against main
Results for commit: e8aa52c Minimum allowed coverage is ♻️ This comment has been updated with latest results |
d3b1e38
to
4505521
Compare
Trivy scanning results. .venv/lib/python3.10/site-packages/PyJWT-2.9.0.dist-info/METADATA (secrets)Total: 1 (MEDIUM: 1, HIGH: 0, CRITICAL: 0) MEDIUM: JWT (jwt-token) .venv/lib/python3.10/site-packages/litellm/llms/huggingface/huggingface_llms_metadata/hf_text_generation_models.txt (secrets)Total: 1 (MEDIUM: 0, HIGH: 0, CRITICAL: 1) CRITICAL: HuggingFace (hugging-face-access-token) .venv/lib/python3.10/site-packages/litellm/proxy/_types.py (secrets)Total: 1 (MEDIUM: 1, HIGH: 0, CRITICAL: 0) MEDIUM: Slack (slack-web-hook) |
packages/ragbits-document-search/src/ragbits/document_search/_main.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/_main.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/_main.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/tests/unit/test_document_search.py
Outdated
Show resolved
Hide resolved
This PR should probably also include some documentation |
@maciejklimek Did you intend to push another commit? Because your comments indicate that you changed things but I don't see the changes here. |
Yeah, sorry for that, but I just really didn't have enough capacity to take care of this. |
@maciejklimek No problem then :) I just thought that maybe you simply forgot to do a push |
- Basic file URI ingestion - Wildcard pattern matching
…main.py Co-authored-by: Ludwik Trammer <[email protected]>
b2a9ee3
to
ce8453a
Compare
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
packages/ragbits-document-search/src/ragbits/document_search/documents/sources.py
Outdated
Show resolved
Hide resolved
…ocuments/sources.py Co-authored-by: Ludwik Trammer <[email protected]>
Solves #221