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

basic streamlit setup with docker #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

machow
Copy link

@machow machow commented Apr 6, 2020

@hugobowne this PR does two main things to address #7:

  • adds streamlit, along with a basic demo from part of your notebook (see app.py file).
  • dockerizes the process, so it can be run using docker-compose up, and previewed from localhost:8000. I commented out some lines that will allow the dashboard to scale better when run on a beefy machine on eg Digital Ocean etc..

screencast of me running the dashboard

Notes

  • it appears there is some uncertainty around how to use conda from docker. I didn't have time to shave the docker+conda yak, so used normal python + pip in the container.
  • the environment.yml did not pin all dependency versions. Added the to requirements.txt.

Hope this helps!

@hugobowne
Copy link

thanks @machow ! We've discussed this & will circle back after Friday Live coding session.

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.

2 participants