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

ReportThread - ModuleNotFoundError #14

Closed
trsavi opened this issue Feb 1, 2022 · 13 comments
Closed

ReportThread - ModuleNotFoundError #14

trsavi opened this issue Feb 1, 2022 · 13 comments

Comments

@trsavi
Copy link

trsavi commented Feb 1, 2022

ReportThread seems to not be working with new version of streamlit (1.5.0.), I got ModuleNotFoundError. When I changed the source code in session_state.py and commented lines for importing report_thread and also disregarded ctx (ctx = ReportThread.get_report_ctx()) variable throughout the script I avoided error on local machine and everything worked fine (I could see basic statistics) but when I deploy it onto streamlit share framework error appeared once again.
Any thoughts on this?

@chanansh
Copy link

chanansh commented Feb 2, 2022

please pull request the necessary changes so we can all enjoy the correction.

@chanansh
Copy link

chanansh commented Feb 2, 2022

@jrieke please help correct this.

@riyadparvez
Copy link

@yanirs
Copy link

yanirs commented Feb 18, 2022

This will be fixed by #16 or #12.

Until one of the PRs is merged, you can install one of the forks, e.g., pip install git+https://github.com/yanirs/streamlit-analytics.

@harishkashyap
Copy link

+1 on this. Same issue on sample code with latest python.

@alaturqua
Copy link

Any plans to fix this soon?

@corticalstack
Copy link

corticalstack commented Apr 25, 2022

+1, same issue

Edit: Found suggestion by Velicanu in link below resolved:

https://discuss.streamlit.io/t/module-not-found-reportthread/5657/6

@TTRh
Copy link

TTRh commented May 13, 2022

any news about this one ?
i saw there is a more recent attempt also here: #18

@standard-outlier
Copy link

This will be fixed by #16 or #12.

Until one of the PRs is merged, you can install one of the forks, e.g., pip install git+https://github.com/yanirs/streamlit-analytics.

Thanks I have tested you code, it works in streamlit 1.4, but getting another import error in 1.8

@franasal
Copy link

@yanirs I tried with your repo but unfortunately, it didn't work. I had to add these modifications to make it work for streamlit 1.8:

main...franasal:streamlit-analytics:main
and added it to my requirements.txt:

git+https://github.com/franasal/streamlit-analytics

@yanirs
Copy link

yanirs commented Jul 28, 2022

Yeah, looks like this project isn't being maintained so I'm not using it. It'll probably break again with new Streamlit versions, unless it's actively maintained.

@fredzannarbor
Copy link

This should either be supported, or be available by default as part of all versions of Streamlit.

@jrieke
Copy link
Owner

jrieke commented Oct 26, 2022

Hey, sorry for the super long delay. I never had time to come around to this. I changed the implementation to work with st.session_state now, so it should be a lot more stable. Check out release 0.4.1, it's tested with Streamlit 1.13.0. (Note that the minimum Streamlit version is now 0.84.0).

@jrieke jrieke closed this as completed Oct 26, 2022
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 a pull request may close this issue.