Fixing the query_batch method of the deepsetcloud document store - …

[zoltan-fedor]

Related Issues

• fixes Code error in the deepsetcloud document store #3722

Proposed Changes:

This is an easy fix of a simple code syntax issue - wrong indentation causing logical error in the code of the deepsetcloud document store's query_batch() method.

How did you test it?

All tests were passing before, so this is tested by code logic only.

Notes for the reviewer

N/A

Checklist

• [ x ] I have read the contributors guidelines and the code of conduct
• [ x ] I have updated the related issue with new insights and changes
• [ x ] I added tests that demonstrate the correct behavior of the change
• [ x ] I've used the conventional commit convention for my PR title
• [ x ] I documented my code
• [ x ] I ran pre-commit hooks and fixed any issue

[tstadel]

LGTM! @zoltan-fedor thx for the fix!

[sjrl]

Hey @zoltan-fedor I think you need to update your branch with the current main to get the build to pass. It looks like your branch is ~265 commits behind the current main branch of Haystack.

[tstadel]

Hey @zoltan-fedor I think you need to update your branch with the current main to get the build to pass. It looks like your branch is ~265 commits behind the current main branch of Haystack.

@zoltan-fedor I updated the branch for you. Hope, that's ok :-)

[zoltan-fedor]

Hey @tstadel , @sjrl
Thanks for fixing this.
I was working on two different PRs in parallel and I must have forked this branch from the wrong base branch.
I was wondering why some of the base CI steps were failing with no numpy... :-)

[ZanSara]

Thank you for this fix! 🙌