Closed shadeMe closed 2 months 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/preprocessors/document_splitter.py | 1 | 98.96% | ||
core/component/component.py | 2 | 97.93% | ||
core/component/sockets.py | 5 | 84.62% | ||
components/retrievers/sentence_window_retriever.py | 10 | 76.09% | ||
core/pipeline/pipeline.py | 13 | 79.55% | ||
<!-- | Total: | 31 | --> |
Totals | |
---|---|
Change from base Build 10522538514: | -0.009% |
Covered Lines: | 7005 |
Relevant Lines: | 7766 |
Related Issues
Proposed Changes:
This PR extends the core component machinery to add support for an
async def async_run(self)
method in components. Theasync_run
method is optional. If it's present, it should have the same parameters (and output types) as therun
method and must be implemented as a coroutine (await
able).How did you test it?
Unit tests
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.