Set up Manual Threading how-to guide is highly problematic #6724
Labels
type: bug
Something isn't correct or isn't working
type: docs
Related to the Panel documentation and examples
Milestone
I've been looking at the Set up Manual Threading multiple times and I think it will cause more harm the good.
import panel as pn
text.c = threading.Condition()
is nescessary I'm not sure the explanation is True. My understanding the Condition enables another thread to wait until a condition. And this is not what we do here. See https://docs.python.org/3/library/threading.html#condition-objects.Maybe use of session life cycle hooks are solution? https://panel.holoviz.org/how_to/callbacks/session.html.
Furthermore the guide only explains how to use manual thread in main
app.py
file that is not shared between sessions. I would also like to see an explanation of using a single shared thread in external module. As in #6723. I think here concerns about performance comes in. Should this thread be pushing updates to all sessions that.depends
on the threads updates? Or should each individual session be running a periodic callback to check for updates? What is most safe and performant?[x] Yes. I'm willing to contribute a PR updating the guide. But I need some guidance and more understanding to be able to do it.
The text was updated successfully, but these errors were encountered: