Future-House / paper-qa

High accuracy RAG for answering questions from scientific documents with citations
Apache License 2.0
6.44k stars 617 forks source link

Removed `tool_names` validation for `gen_answer` being present #685

Closed jamesbraza closed 1 week ago

jamesbraza commented 1 week ago

If any of the tools are excluded, PaperQA will not function. They all play a distinct and required role (paper_search to get new papers, gather_evidence to get relevant evidence, and gen_answer for answering), and none are optional.

We can either:

We can YAGNI on the first route, so this PR takes the second route as suggested in this PR comment: https://github.com/Future-House/paper-qa/pull/684#discussion_r1841332919