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://gpuweb.github.io/gpuweb/#abstract-opdef-validating-gpuprogrammablestage>

            https://github.com/gpuweb/gpuweb/pull/4387
            <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
            <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
            
<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

            <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
            <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
            <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
            <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/6173453d-3384-4516-9f32-cc4eaf085d94%40chromium.org.

Reply via email to