refactor: utility function for docstore deserialization

deepset-ai / haystack

AI orchestration framework to build customizable, production-ready LLM applications. Connect components (models, vector DBs, file converters) to pipelines or agents that can interact with your data. With advanced retrieval methods, it's best suited for building RAG, question answering, semantic search or conversational agent chatbots.

Apache License 2.0

17.72k stars 1.92k forks source link

Related Issues

follow up of #8207

Proposed Changes:

Introduce a utility function to perform this common operation
test it properly

How did you test it?

CI, new tests

Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
I documented my code
I ran pre-commit hooks and fixed any issue

Pull Request Test Coverage Report for Build 10386461988

Details

0 of 0 changed or added relevant lines in 0 files are covered.

7 unchanged lines in 1 file lost coverage.

Overall coverage increased (+0.05%) to 90.134%

Files with Coverage Reduction	New Missed Lines	%
components/retrievers/sentence_window_retriever.py	7	79.55%
<!--	Total:	7		-->

Files with Coverage Reduction

New Missed Lines

components/retrievers/sentence_window_retriever.py

79.55%

<!--

Total:

-->

Totals
Change from base Build 10384533702:	0.05%
Covered Lines:	6943
Relevant Lines:	7703

Totals

Change from base Build 10384533702:

0.05%

Covered Lines:

6943

Relevant Lines:

7703

deepset-ai / haystack