LGTM3 On Saturday, November 25, 2023 at 7:08:21 PM UTC+1 Mike Taylor wrote:
> LGTM2 > On 11/24/23 8:53 AM, Rick Byers wrote: > > Thanks, good enough for me! LGTM1 > > On Fri, Nov 24, 2023 at 8:32 AM François Beaufort <fbeauf...@google.com> > wrote: > >> >> >> On Fri, Nov 24, 2023 at 1:46 AM Rick Byers <rby...@chromium.org> wrote: >> >>> Seems trivial to me, just one little question: >>> >>> On Thu, Nov 23, 2023 at 10:34 AM 'François Beaufort' via blink-dev < >>> blink-dev@chromium.org> wrote: >>> >>>> Contact emails >>>> >>>> fbeauf...@google.com, cwal...@google.com >>>> >>>> Explainer >>>> >>>> https://github.com/gpuweb/gpuweb/issues/4342 >>>> >>>> Specification >>>> >>>> >>>> https://gpuweb.github.io/gpuweb/#abstract-opdef-validating-gpuprogrammablestage >>>> >>>> https://github.com/gpuweb/gpuweb/pull/4387 >>>> >>>> Summary >>>> >>>> WebGPU now allows developers to omit entry points to shader modules >>>> when creating a pipeline, improving ergonomics. If no default entry point >>>> is found, a GPUValidationError will be triggered as usual. >>>> >>>> Blink component >>>> >>>> Blink>WebGPU >>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EWebGPU> >>>> >>>> TAG review >>>> >>>> WebGPU spec: https://github.com/w3ctag/design-reviews/issues/626 >>>> >>>> TAG review status >>>> >>>> A TAG review was not deemed necessary for this minor addition to the >>>> WebGPU spec, as it adheres to established conventions. >>>> >>>> Risks >>>> >>>> Interoperability and Compatibility >>>> >>>> Default entry points to shader modules have not yet been implemented in >>>> any browser, but have been approved by the GPU for the Web Community >>>> Group, >>>> with representatives from Chrome, Firefox, and Safari. See >>>> https://github.com/gpuweb/gpuweb/wiki/GPU-Web-2023-11-08#default-entrypoints-to-shader-modules-4342-corentin-is-very-excited-about-this-tiny-addition >>>> >>>> Gecko: Positive ( >>>> https://github.com/mozilla/standards-positions/issues/931) >>>> >>>> WebKit: Positive ( >>>> https://github.com/WebKit/standards-positions/issues/284) >>>> >>> >>>> >>> >>>> Web developers: Positive. https://github.com/gpuweb/gpuweb/issues/4342 >>>> >>>> Other signals: >>>> >>>> WebView application risks >>>> >>>> Does this intent deprecate or change behavior of existing APIs, such >>>> that it has potentially high risk for Android WebView-based applications? >>>> >>>> None at the moment, WebGPU currently does not ship on Android WebView. >>>> >>>> >>>> Debuggability >>>> >>>> No DevTools changes are required. >>>> >>>> Will this feature be supported on all six Blink platforms (Windows, >>>> Mac, Linux, Chrome OS, Android, and Android WebView)? >>>> >>>> No >>>> >>>> All platforms will eventually have support. Will immediately be >>>> available on Android, ChromeOS, Mac, and Windows, since those platforms >>>> already support WebGPU. Linux is planned to have WebGPU support in the >>>> future, so this feature will become available when WebGPU does. >>>> >>>> Is this feature fully tested by web-platform-tests >>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>> ? >>>> >>>> Work-in-progress. >>>> >>>> WebGPU/WGSL have a conformance test suite ( >>>> https://github.com/gpuweb/cts) that is regularly pulled into Chromium >>>> and part of the testing of Dawn/Tint in Chromium. >>>> >>> >>> So does this addition have coverage in the WebGPU CTS? I went looking >>> through the CTS and WebGPU spec repo to try to understand the typical >>> working mode. I see >>> <https://github.com/gpuweb/cts#webgpu-conformance-test-suite> that the >>> test suite is considered normative and any mismatch between the CTS and >>> specs are bugs, so does that mean that spec changes are expected to >>> come along with >>> <https://github.com/foolip/testing-in-standards/blob/master/lifecycle.md> >>> test changes too (as is the practice in many other groups now)? >>> >>> Regardless, chromium's policy is that we expect conformance test >>> coverage (WPT or otherwise) for every I2S or an explanation of why such >>> test coverage isn't currently practical. >>> >> >> I've just sent a CTS PR at https://github.com/gpuweb/cts/pull/3190 for >> this specific feature. >> >> >>> >>> Flag name on chrome://flags >>>> >>>> None >>>> >>>> Finch feature name >>>> >>>> None >>>> >>>> Non-finch justification >>>> >>>> None >>>> >>>> Requires code in //chrome? >>>> >>>> False >>>> >>>> Tracking bug >>>> >>>> https://bugs.chromium.org/p/dawn/issues/detail?id=2254 >>>> >>>> Availability expectation >>>> >>>> Feature is available only in Chromium browsers for the near future, on >>>> the order of months. Other browsers intend to ship WebGPU support, but >>>> don't have specified timelines. >>>> >>>> Non-OSS dependencies >>>> >>>> Does the feature depend on any code or APIs outside the Chromium open >>>> source repository and its open-source dependencies to function? >>>> >>>> No >>>> >>>> Estimated milestones >>>> >>>> Shipping on desktop >>>> >>>> 121 >>>> >>>> Shipping on Android >>>> >>>> 121 >>>> >>>> Anticipated spec changes >>>> >>>> Open questions about a feature may be a source of future web compat or >>>> interop issues. Please list open issues (e.g. links to known github issues >>>> in the project for the feature specification) whose resolution may >>>> introduce web compat/interop risk (e.g., changing to naming or structure >>>> of >>>> the API in a non-backward-compatible way). >>>> >>>> None >>>> >>>> Link to entry on the Chrome Platform Status >>>> >>>> https://chromestatus.com/feature/5102870756327424 >>>> >>>> This intent message was generated by Chrome Platform Status >>>> <https://chromestatus.com/>. >>>> >>>> >>>> -- >>>> You received this message because you are subscribed to the Google >>>> Groups "blink-dev" group. >>>> To unsubscribe from this group and stop receiving emails from it, send >>>> an email to blink-dev+unsubscr...@chromium.org. >>>> To view this discussion on the web visit >>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAPpwU5KPXpmcrMCz9keGzv88zFEeX_rgujaN63mr2aHe2nydFA%40mail.gmail.com >>>> >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAPpwU5KPXpmcrMCz9keGzv88zFEeX_rgujaN63mr2aHe2nydFA%40mail.gmail.com?utm_medium=email&utm_source=footer> >>>> . >>>> >>> -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY_s-DapFcw63KpQbAnjwtc-LMPA1j3GaifVMNWnCXLEPg%40mail.gmail.com > > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY_s-DapFcw63KpQbAnjwtc-LMPA1j3GaifVMNWnCXLEPg%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > > -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/98d58021-aa5a-4c71-821e-40ebd5ff4568n%40chromium.org.