-
Notifications
You must be signed in to change notification settings - Fork 48
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
Specify WebNN timelines #529
Comments
From what I gather, we have at-least 3 timelines:
WebNN is similar to WebGL's programming model in this aspect, |
Is it correct to say that the current standard prose on parallelism is enough to capture timelines?
Do we need to define a specialized term for timelines? If we do, we should also define the relationships vs context, graph etc.:
From the app script point of view, what is the minimal differentiation of terms we need to do? EDIT: I see that it comes from WebGPU timelines. |
Not fully. We still need to define what state gets exposed per API operation. For example,
WebNN could map to WebGPU timelines when the deviceType is GPU but not necessarily for the other device types. |
Building on what @zolkis mentioned above, as a non-WebGPU expert, it would be nice for the eventual spec text to explain the relationship between timelines and event loops / task sources/task queues . These are how most web specs represent cross-thread / cross-process work. Maybe when dealing with WebGPU devices a WebNN timeline is a WebGPU timeline, but otherwise is a task queue... I don't know enough to know if that's sufficient, but want to learn! |
The spec mentions some timelines (a "parallel timeline", "a GPU timeline", "a different timeline", "the offloaded timeline", etc...) but these timelines are not described anywhere. Meanwhile, #482 mentions a "device timeline" and "content timeline" (and, at the time of writing, there have been early discussions about whether an
MLQueue
is needed - which may or may not require a "queue timeline", as well)These timelines should be clearly defined, including:
The text was updated successfully, but these errors were encountered: