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

[FEATURE] Add a DocumentReady event #10265

Closed
philippjfr opened this issue Jul 7, 2020 · 4 comments · Fixed by #10266
Closed

[FEATURE] Add a DocumentReady event #10265

philippjfr opened this issue Jul 7, 2020 · 4 comments · Fixed by #10266

Comments

@philippjfr
Copy link
Contributor

In many cases you want your application to display something as quickly as possible. In the current design of bokeh server everything has to be initialized in the initial request. One solution to alleviate this issue would be to defer any expensive computation on initialization until after we have actually rendered something to screen. However currently there is no way to tell when the frontend is done initializing the rendered Document.

Therefore I propose adding a BokehEvent which signals when the Document is ready (or in BokehJS terminology, when it is idle).

@bryevdv
Copy link
Member

bryevdv commented Aug 18, 2020

Noting that the work in #10266 was only for a Python/Bokeh server event, not yet for js_on_event type event usable with CustomJS (e.g. in a standalone document)

@philippjfr
Copy link
Contributor Author

Right, should have checked this but a Document does not yet have the ability to register js callbacks on it.

@bryevdv
Copy link
Member

bryevdv commented Sep 18, 2020

@philippjfr this issue got closed is there a (new?) issue for a JS init event? (I actually thought there was a different older issue about this than this one but I can't find it if so)

@kszlim
Copy link

kszlim commented Jan 21, 2021

@philippjfr I was wondering the same thing, I was trying to register a JS callback on this event, but didn't find any documentation that implied it didn't work until I came across this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants