Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/improve streamlit UI #2

Merged
merged 13 commits into from
Sep 13, 2023

Conversation

ahmedbesbes
Copy link
Contributor

Hello @zainhoda

Nice work you did on the app 馃憤馃徎 !

I took the time to explore it and played with Vanna a bit and I'm really impressed with its capabilities.

As discussed, I forked the repo and worked on the following features in this PR:

  • I refactored the code
  • I added caching for the connection setup
  • I added the possibility to run the app locally by using .env files
  • I added poetry as a package manager
  • I added the possibility of halting the execution of the flow and manually fixing the generated SQL and Plotly code (demo in updated README.md file)

@zainhoda zainhoda merged commit 877003a into vanna-ai:main Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants