-
Notifications
You must be signed in to change notification settings - Fork 33
Conversation
I have a preliminary fix available in
|
Just manually tested this on the |
No luck yet for me @chriddyp. LMK if you want to see my script to help debug (or decide it's a different issue) |
@wtluke - Yeah, that would be helpful. Thanks! |
Emailed 👍 |
@wtluke - It looks like the infinite looping was a separate issue that was caused by having 2 Applying this patch to your example seemed to reduce the number of times one of your callbacks was being called from 16 times to just 2 times 🚀 ! |
Wow. Wow. Thank you, Chris. And this fix is insane - it has transformed the app into something unbelievably fast! Has caused me a lot of pain (trying to work out ways of avoiding 16 callbacks) - I never realised your solution here (calling it two times, per output) would have been possible. Wish I'd spoken up earlier. Absolutely superb - can't thank you enough. Onwards! |
following instructions from https://percy.io/docs/learn/parallel-tests and http://tox.readthedocs.io/en/latest/example/basic.html#setting-environme nt-variables
`test_race_condtions` doesn’t make any snapshots
1 for each subclass (even if no snapshots were made)
Fixes issue brought up in https://community.plot.ly/t/callback-is-fired-excessively-in-multi-app-setup-once-for-each-input-instead-of-once-for-each-callback/5789/4