defog-ai / sql-eval

Evaluate the accuracy of LLM generated outputs
Apache License 2.0
485 stars 52 forks source link

Improve psycopg dialect string handling #139

Closed wongjingping closed 2 months ago

wongjingping commented 2 months ago

We would silently fail and hide the exceptions whenever the environment doesn't have psycopg/psycopg2 installed. This doesn't usually happened on a machine that has our usual setup with everything pre-installed, but could happen in new environments or on other users' environments. We make the error handling/messages more explicit so that people know if there's an error and how to remedy.

rishsriv commented 2 months ago

Thank you!