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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add example for Streamlit app that uses PostgreSQL #262

Merged
merged 19 commits into from
Aug 3, 2024

Conversation

stanleyo03
Copy link
Contributor

@stanleyo03 stanleyo03 commented Aug 1, 2024

Link to app on Ploomber Cloud: https://twilight-fog-4463.ploomberapp.io/

Closes #258


📚 Documentation preview 📚: https://ploomber-doc--262.org.readthedocs.build/en/262/

Copy link
Contributor

@bryannho bryannho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Basic function works well - one question.

When I enter a new data point in, I notice that I have to change one of the axes for the new point to be loaded on the chart and table (I assume because the frontend doesn't grab the new data until an axis is changed). This might be confusing since users expect to see it immediately without doing anything.

Can you add an automatic reload? You could have it reload after a delay (the postgres query should go through pretty fast) or worst case, add a refresh button to show the changes without changing any axes.

@neelasha23
Copy link
Contributor

This is not restarting for some reason:
Screenshot 2024-08-02 at 5 47 28 PM

Can u please start the app and lmk @stanleyo03

@bryannho
Copy link
Contributor

bryannho commented Aug 2, 2024

This is not restarting for some reason:

Screenshot 2024-08-02 at 5 47 28 PM

Can u please start the app and lmk @stanleyo03

This is a bug on the backend. @stanleyo03 you probably need to delete this one and re deploy it as a new app

@stanleyo03
Copy link
Contributor Author

stanleyo03 commented Aug 2, 2024

Copy link
Contributor

@bryannho bryannho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

App looks good with the reload! One minor comment

examples/streamlit/postgres-connection/README.md Outdated Show resolved Hide resolved
@neelasha23
Copy link
Contributor

In the left sidebar, please add a very brief description of the utility of the app and what happens when a user inputs a new point (what changes should they expect to see) @stanleyo03

@stanleyo03
Copy link
Contributor Author

@bryannho @neelasha23 should be ready for merge

@neelasha23 neelasha23 merged commit f34af85 into main Aug 3, 2024
1 check passed
@neelasha23 neelasha23 deleted the streamlit_postgres_example branch August 3, 2024 16:43
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.

add example for connection streamlit to postgres
3 participants