-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chat with CSV fix #160
Chat with CSV fix #160
Conversation
Deployed without OpenAI key: https://calm-pond-0229.ploomberapp.io/ |
Approach 1: (works in Jupyterlab but not Voila) If OpenAI key not set then display input box where users can input the key. It will get saved and run all the below cells again to display the chatbox by running:
The last cell re-running is needed because there's an This works in Jupyterlab: Screen.Recording.2024-03-25.at.5.22.22.PM.movBut this doesn't work in Voila, found a related issue: voila-dashboards/voila#614 Screen.Recording.2024-03-25.at.5.21.31.PM.movApproach 2 (Works): Have removed the
This will create the chat input box irrespective of whether OpenAI API key is found or not. But it would be disabled till user inputs the API Key. This would eliminate the need to rerun this cell. And user will be prompted for entering the API key when uploading data or using sample data. I was also getting an error when uploading a file, fixed it by changing this line to Deployed apps: Please review @edublancas |
@@ -1,4 +1,5 @@ | |||
# requirements for chat-with-csv | |||
sqlalchemy==2.0.28 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest version of sqlalchemy is breaking DuckDB
Closes #152
📚 Documentation preview 📚: https://ploomber-doc--160.org.readthedocs.build/en/160/