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: adapted to the latested streamlit version 1.23.1 #2

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

zhangzh-pku
Copy link

No description provided.

@Wauplin
Copy link
Owner

Wauplin commented Jun 6, 2023

Hi @zhangzh-pku thanks for opening a PR. First of all, sorry for the mess on this PR. I wanted to quickly fix CI issues from the browser but I'll to dig a bit more into it. This was a hobby project for me so I'll need to find some time to understand what I did at that time :D

In any case, I'm glad to see it's still used. What's your use case for streamlit_sync? :)

@zhangzh-pku
Copy link
Author

Hi @zhangzh-pku thanks for opening a PR. First of all, sorry for the mess on this PR. I wanted to quickly fix CI issues from the browser but I'll to dig a bit more into it. This was a hobby project for me so I'll need to find some time to understand what I did at that time :D

In any case, I'm glad to see it's still used. What's your use case for streamlit_sync? :)

Sorry for the slow reply, I am currently dealing with a scenario where resource loading is relatively heavy, and it's connected to a backend task that takes a long time to process. Therefore, I hope to log some context information about the connection, so I can revisit this task later on.

@wirelessr
Copy link

Upvote for this PR.

My use case is a multi-client service, and the status of each client is stored in backend database; however, I don't want to reload every status from the database frequently. Therefore, the perfect match is stored those status in st.session_state to share between each other.

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.

3 participants