search

Future-House / paper-qa

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

Unsilencing manifest read failure when no CSV header is present #587

Closed jamesbraza closed 1 month ago

jamesbraza commented 1 month ago

It turns out there were two problems with our manifest loading:

  1. If no CSV header is present, our maybe_get_manifest silently fails
  2. It used logging.error, which bypasses our logging handlers

This PR fixes both issues