ploomber / sql

https://ploomber-sql.readthedocs.io
Apache License 2.0
17 stars 5 forks source link

Voila Dashboard #79

Closed TegveerG closed 1 year ago

TegveerG commented 1 year ago

Describe your changes

draft voila-pp.ipynb - interactive tables and 2 seaborn plots with widgets and insights complete for dashboard fixed cells in explore.ipynb

Issue number

Closes #77

Checklist before requesting a review


:books: Documentation preview :books:: https://ploomber-sql--79.org.readthedocs.build/en/79/

lfunderburk commented 1 year ago

Merge main into your branch

lfunderburk commented 1 year ago

Hi Tegveer,

Implemented menu, see the voila-app-copy.ipynb and menu.py: I focused only on the table all vehicles,

I still feel that showing three tables is overkill,so I created an additional menu to pick fuel based,electric or hybrid vehicles

https://github.com/ploomber/sql/assets/13559004/b2494ccf-eed6-4be7-8bab-400381ef3cd0

TegveerG commented 1 year ago

Hi Tegveer,

Implemented menu, see the voila-app-copy.ipynb and menu.py: I focused only on the table all vehicles,

I still feel that showing three tables is overkill,so I created an additional menu to pick fuel based,electric or hybrid vehicles

sql-voila.mov

Yes, a single table is better, this is wonderful!

TegveerG commented 1 year ago

Hi @TegveerG

Great work!

The only odd thing is when I run the voila app it hangs at cell 13. What version of JupySQl are you using?

I have mixed feelings about the multiple "display code" Maybe put them all in one cell instead so we can appreciate what is going on?

Hi @lfunderburk, am using JupySQL==0.9.0 Jonathan just tested out the dashboard on his local and it renders well. Have added the function with display() in the same cell as interactive_output() in the new commit