-
-
Notifications
You must be signed in to change notification settings - Fork 525
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 autoreload option #1983
Add autoreload option #1983
Conversation
I want this so badly as it will speed up my workflow a lot. Does it only reload one module or all modules that change? |
Currently reloads the scripts you are serving but I'll be working on making that configurable. |
You should consider whether more advanced reloading is worth it. I know from Streamlit that reloading nested modules can be difficult. They never got it working 100% even though it was one of the first bugs I reported. |
Can you clarify what you mean by "nested modules"? |
You can see a description of some of the problems here streamlit/streamlit#358 |
I've now borrowed some code from streamlit after trying a couple of different approaches but I suspect we'll have the same limitations as they do. |
Codecov Report
@@ Coverage Diff @@
## master #1983 +/- ##
==========================================
- Coverage 84.90% 84.71% -0.20%
==========================================
Files 160 162 +2
Lines 19522 19696 +174
==========================================
+ Hits 16576 16686 +110
- Misses 2946 3010 +64
Continue to review full report at Codecov.
|
Adds an
--autoreload
option topanel serve
which reloads the application when a change in a script/notebook is detected.out.mp4