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

Document that ProcessEvents is only needed for AllowProcessEvents callbacks #498

Open
Tracked by #502
kainino0x opened this issue Jan 15, 2025 · 1 comment
Open
Tracked by #502

Comments

@kainino0x
Copy link
Collaborator

We should explain in docs for wgpuInstanceProcessEvents that it is only needed for AllowProcessEvents callbacks (that aren't waited), or in other words, you don't need it if every future either is waited or has AllowSpontaneous.

(Which led me to another question: is it fine if my app never calls wgpuInstanceProcessEvents or wgpuDeviceTick? If it is not, when should they be called?)

Originally posted by @zcbenz in #491 (comment)

@lokokung
Copy link
Collaborator

Agreed! It's important to note that although tehnically we shouldn't need to call ProcessEvents or Tick if all callbacks are AllowSpontaneous, apps that do not continually submit work to the queue (in native) may not see forward progress if they don't call ProcessEvents or Tick at the moment. This will likely be an issue until we resolve a good way to make forward progress in another thread or something.

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

No branches or pull requests

2 participants