Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Kai Ninomiya
As one last update for now, we've changed the CL to completely remove the feature. This way if we do succeed in merging to 113 the deprecated feature will not ship to stable, so the compat risk of removing it will be zero. Updating chromestatus. -Kai (he/they) On Wed, Apr 19, 2023 at 1:18 PM Ric

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Rick Byers
Perfect, thank you! Good luck! On Wed, Apr 19, 2023 at 4:10 PM Kai Ninomiya wrote: > Thanks! Loko has filed https://crbug.com/1434745 for the merge request > and I've filed https://crbug.com/1434766 as a releaseblock for 114 to > track anything we need to do if it doesn't get merged back. > > I

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread 'Kai Ninomiya' via blink-dev
Thanks! Loko has filed https://crbug.com/1434745 for the merge request and I've filed https://crbug.com/1434766 as a releaseblock for 114 to track anything we need to do if it doesn't get merged back. -Kai On Wednesday, April 19, 2023 at 10:08:53 AM UTC-7 Rick Byers wrote: > Ah, good point :-)

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Kai Ninomiya
Thanks! Loko has filed https://crbug.com/1434745 for the merge request and I've filed https://crbug.com/1434766 as a releaseblock for 114 to track anything we need to do if it doesn't get merged back. I also fixed the GPUSupporteFeatures/Limits typos in the chromestatus page. -Kai (he/they) On

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Rick Byers
Ah, good point :-) I just approved the CL for the purposes of attempting a merge to M113. Please follow up here on whether that happened or if we need to proceed with the breaking change for M114. Rick On Wed, Apr 19, 2023 at 12

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Kai Ninomiya
Apologies, I made an error in the I2S, this is in an attribute in GPUSupportedLimits (an interface), not GPUSupportedFeatures (a set). Hence considering it a breaking change. We have been considering a request to merge back to M113 but given we can't land on main without LGTM on an I2S, and we can

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Rick Byers
Given how close we are to M113 stable release, it may be too late for a merge back. If so, I imagine removal should be pretty low risk and easy but given all the trouble we've had lately I do think we need to follow the breaking change process, including putting the changes behind a flag

Re: [blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-19 Thread Yoav Weiss
On Tue, Apr 18, 2023 at 8:32 PM Kai Ninomiya wrote: > (sending from correct email address this time) > Due to the nature of this removal I've elected to send this as an Intent > to Ship because we are seeking LGTMs. Please let me know if this is > inappropriate. > Contact emailskain...@chromium.o

[blink-dev] Fwd: Intent to Ship: Remove WebGPU limit maxFragmentCombinedOutputResources

2023-04-18 Thread Kai Ninomiya
(sending from correct email address this time) Due to the nature of this removal I've elected to send this as an Intent to Ship because we are seeking LGTMs. Please let me know if this is inappropriate. Contact emailskain...@chromium.org Explainerhttps://github.com/gpuweb/gpuweb/issues/4018 Speci