Found and squished a bug in our Q&A process: we were not passing all documents in chat context to the Q&A LLM, as the default documents to retrieve from is 4 and we were not overriding this. This caused an odd symptom in which the bot was seemingly unable to answer a question about employees or salaries in full - instead one needed to piece together a complete answer from several consecutive questions, and even then you'd not know you were in receipt of the complete answer unless you knew what you were looking for.
The fix is simple, though there are related test changes, plus an incorrect comment and lint-ignore statement were also tackled here.
Description
Found and squished a bug in our Q&A process: we were not passing all documents in chat context to the Q&A LLM, as the default documents to retrieve from is 4 and we were not overriding this. This caused an odd symptom in which the bot was seemingly unable to answer a question about employees or salaries in full - instead one needed to piece together a complete answer from several consecutive questions, and even then you'd not know you were in receipt of the complete answer unless you knew what you were looking for.
The fix is simple, though there are related test changes, plus an incorrect comment and lint-ignore statement were also tackled here.
Resolves #902
Screenshots
All salary info found in one call:
Checklist
Have you done the following?