Closed sjrl closed 1 month ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
components/embedders/sentence_transformers_document_embedder.py | 2 | 96.49% | ||
components/embedders/sentence_transformers_text_embedder.py | 2 | 95.92% | ||
components/rankers/transformers_similarity.py | 6 | 91.11% | ||
components/retrievers/sentence_window_retriever.py | 9 | 83.33% | ||
components/fetchers/link_content.py | 13 | 79.79% | ||
<!-- | Total: | 32 | --> |
Totals | |
---|---|
Change from base Build 10197271394: | 0.02% |
Covered Lines: | 6883 |
Relevant Lines: | 7638 |
Hey @dfokina let me know if the docstrings for the new parameters look okay!
Related Issues
Proposed Changes:
Adds
model_kwargs
andtokenizer_kwargs
to the componentsTransformersSimilarityRanker
,SentenceTransformersDocumentEmbedder
,SentenceTransformersTextEmbedder
How did you test it?
Expanded existing unit tests
Notes for the reviewer
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.