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

Update Status of this document for CR Snapshot #532

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Update Status of this document for CR Snapshot #532

merged 1 commit into from
Feb 13, 2024

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Jan 26, 2024

This status text is a high-level summary for busy people.

  • Implementation experience update
  • Significant new & removed features, use cases, conventions update
  • Significant new features in development, MLBuffer

Preview | Diff

@anssiko
Copy link
Member Author

anssiko commented Jan 26, 2024

This PR pre-empts the group addresses #531 as discussed in https://www.w3.org/2024/01/25-webmachinelearning-minutes.html#t05

@anssiko anssiko requested a review from dontcallmedom January 26, 2024 09:24
@anssiko
Copy link
Member Author

anssiko commented Jan 26, 2024

@wchao1115 @huningxin also PTAL

This is crafted based on our discussion in https://www.w3.org/2024/01/25-webmachinelearning-minutes.html#t06 with consideration for "what's new" since initial CR March 2023 and what people outside this group should know given an opportunity to do an elevator pitch to them. This means we need to abstract the details.

index.bs Outdated
integration. A proposal to
<a href="https://github.com/webmachinelearning/webnn/pull/322">simplify
MLContext creation</a> is being discussed. This document is maintained and
Since the <a href="https://www.w3.org/TR/2023/CR-webnn-20230330/">initial Candidate Recommendation Snapshot</a> the Working Group has gathered further <a href="https://webmachinelearning.github.io/webnn-status/">implementation experience</a> and added new operations and data types needed for well-known <a href="https://github.com/webmachinelearning/webnn/issues/375">transformers to support generative AI use cases</a>. In addition, the group has removed select features informed by this implementation experience: higher-level operations that can be expressed in terms of lower-level primitives in a performant manner, and support for synchronous execution. The group has also updated the specification to use modern authoring conventions to improve interoperability and precision of normative definitions and is developing a new feature, a <a href="https://github.com/webmachinelearning/webnn/issues/482">backend-agnostic storage type</a>, to improve performance and interoperability between the WebNN, WebGPU APIs and purpose-built hardware for ML.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Should we expand a bit more on what 'further implementation experience' is? i.e. should we be more specific about the ongoing work in the Chromium code base across Google, Microsoft, and Intel, etc.?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @wchao1115 for your review. It may be hard to succinctly expand 'further implementation experience' in the limited space we have in this status section.

For this reason, we direct interested readers to https://webmachinelearning.github.io/webnn-status/ that also provides further pointers to Web Platform Tests. WPT is a resource W3C stakeholders are interested in in particular when they evaluate implementation experience. I think we are good in this respect with the materials we have.

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
@anssiko
Copy link
Member Author

anssiko commented Feb 8, 2024

@dontcallmedom PTAL, PR updated to note MLCommandEncoder removal.

index.bs Outdated
integration. A proposal to
<a href="https://github.com/webmachinelearning/webnn/pull/322">simplify
MLContext creation</a> is being discussed. This document is maintained and
Since the <a href="https://www.w3.org/TR/2023/CR-webnn-20230330/">initial Candidate Recommendation Snapshot</a> the Working Group has gathered further <a href="https://webmachinelearning.github.io/webnn-status/">implementation experience</a> and added new operations and data types needed for well-known <a href="https://github.com/webmachinelearning/webnn/issues/375">transformers to support generative AI use cases</a>. In addition, informed by this implementation experience, the group removed <code>MLCommandEncoder</code></a>, support for synchronous execution, and higher-level operations that can be expressed in terms of lower-level primitives in a performant manner. The group has also updated the specification to use modern authoring conventions to improve interoperability and precision of normative definitions and is developing a new feature, a <a href="https://github.com/webmachinelearning/webnn/issues/482">backend-agnostic storage type</a>, to improve performance and interoperability between the WebNN, WebGPU APIs and purpose-built hardware for ML.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking some more about the backend-agnostic storage new feature - I think we should indicate whether we expect to republish a CR Snapshot if/when that feature lands (vs keeping it for a potential post-Recommendation iteration); I'm happy to propose some suggested language if you confirm the former is our plan.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe we don’t want to rush to post-CR stages, especially considering more implementers just joined the WG. I’d expect us to do another CRS when this new feature has matured adequately.

Contributions welcome.

index.bs Outdated Show resolved Hide resolved
@anssiko anssiko force-pushed the cr2-sotd branch 2 times, most recently from 680c17a to 15168d3 Compare February 12, 2024 10:22
This status text is a high-level summary for busy people.

- Implementation experience update
- Significant new & removed features, use cases, conventions update
- Significant new features in development, MLBuffer

Co-authored-by: Dominique Hazael-Massieux <[email protected]>
@anssiko
Copy link
Member Author

anssiko commented Feb 13, 2024

@dontcallmedom I'll merge this SOTD update now even if we're not going to publish immediately.

I feel it is a good practice to have this section reflect reality, and especially new participants might glimpse here for the latest high-level status. We should pick a habit to maintain this on an ongoing basis, not just for major milestones.

@anssiko anssiko merged commit d48e394 into main Feb 13, 2024
2 checks passed
@anssiko anssiko deleted the cr2-sotd branch February 13, 2024 09:21
github-actions bot added a commit that referenced this pull request Feb 13, 2024
SHA: d48e394
Reason: push, by anssiko

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

4 participants