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

feat: start Streamlit Dockerfile process #69

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sfc-gh-twhite
Copy link
Collaborator

@sfc-gh-twhite sfc-gh-twhite commented Nov 27, 2024

We should have a Dockerfile for people to use as a reference for getting this up and running in SPCS. We will need to ensure this works locally and then provide instructions for configuring and deploying the image to Snowflake.

I pulled the initial example for the file from the Streamlit docs.

@sfc-gh-twhite sfc-gh-twhite marked this pull request as ready for review December 2, 2024 21:47
@sfc-gh-twhite sfc-gh-twhite marked this pull request as draft December 2, 2024 21:48
@sfc-gh-twhite
Copy link
Collaborator Author

sfc-gh-twhite commented Dec 2, 2024

The Streamlit app currently expects environment variables to be set and is looking for a NEWS_API_TOKEN environment variable. I think we should make this more generic and explore an alternative connection configuring strategy (possibly using connections.toml).

I am tracking this effort with #74.

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.

feat(app): rewrite demo application with connection config tweaks and use generic PythonTool
1 participant