search

RamiAwar / dataline

Chat with your data - AI data analysis and visualization on CSV, Postgres, MySQL, Snowflake, SQLite...
https://dataline.app
GNU General Public License v3.0
129 stars 6 forks source link

feat: Support error streaming to FE #223

Closed RamiAwar closed 1 week ago

RamiAwar commented 1 week ago

Closes Closes #216, #175

Tested thoroughly by deleting all keys, imposing rate limits, etc.

Pasted image 20240624212828
RamiAwar commented 1 week ago

All good, but maybe we should add a general except (perhaps in generate_with_errors) for potential errors we aren't handling yet? Also check my comment on forward_connection_errors

Summary of our convo: I think if it's not UserFacing, it should be an internal server error (500). That's already handled differently by the FE (error querying assistant error)