Reduce default maxSubcasesInFlight instead of using batching (#4006)

I forgot that we could adjust maxSubcasesInFlight to avoid problems with there being too many subcases in a case. This reverts commit ec54937 as well as changes the default value for maxSubcasesInFlight.
gpuweb · Oct 16, 2024 · ecefa0d · ecefa0d
1 parent 648e056
commit ecefa0d
Show file tree

Hide file tree

Showing 66 changed files with 111 additions and 175 deletions.
diff --git a/src/common/framework/test_config.ts b/src/common/framework/test_config.ts
@@ -40,7 +40,7 @@ export type TestConfig = {
 
 export const globalTestConfig: TestConfig = {
   enableDebugLogs: false,
-  maxSubcasesInFlight: 500,
+  maxSubcasesInFlight: 100,
   testHeartbeatCallback: () => {},
   noRaceWithRejectOnTimeout: false,
   unrollConstEvalLoops: false,

diff --git a/src/resources/cache/hashes.json b/src/resources/cache/hashes.json
diff --git a/src/webgpu/shader/validation/expression/call/builtin/abs.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/abs.spec.ts
@@ -24,7 +24,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kAllNumericScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/acos.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/acos.spec.ts
@@ -28,7 +28,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/acosh.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/acosh.spec.ts
@@ -28,7 +28,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/asin.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/asin.spec.ts
@@ -28,7 +28,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/asinh.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/asinh.spec.ts
@@ -34,7 +34,6 @@ const additionalRangeForType = rangeForType(
 );
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/atan.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/atan.spec.ts
@@ -27,7 +27,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/atan2.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/atan2.spec.ts
@@ -31,7 +31,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/atanh.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/atanh.spec.ts
@@ -28,7 +28,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/ceil.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/ceil.spec.ts
@@ -25,7 +25,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/clamp.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/clamp.spec.ts
@@ -29,7 +29,6 @@ const kValuesTypes = objectsToRecord([
 ]);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/cos.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/cos.spec.ts
@@ -27,7 +27,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/cosh.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/cosh.spec.ts
@@ -25,7 +25,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/countLeadingZeros.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/countLeadingZeros.spec.ts
@@ -24,7 +24,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConcreteIntegerScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/countOneBits.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/countOneBits.spec.ts
@@ -24,7 +24,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConcreteIntegerScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/countTrailingZeros.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/countTrailingZeros.spec.ts
@@ -24,7 +24,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConcreteIntegerScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/cross.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/cross.spec.ts
@@ -37,7 +37,6 @@ function quantizeFunctionForScalarType(type: ScalarType): QuantizeFunc<number> {
 }
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/degrees.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/degrees.spec.ts
@@ -26,7 +26,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValuesTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() inputs rejects invalid values

diff --git a/src/webgpu/shader/validation/expression/call/builtin/distance.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/distance.spec.ts
@@ -25,7 +25,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValidArgumentTypes = objectsToRecord(kConvertableToFloatScalarsAndVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/dot.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/dot.spec.ts
@@ -21,7 +21,6 @@ export const g = makeTestGroup(ShaderValidationTest);
 const kValidArgumentTypes = objectsToRecord(kConvertableToFloatVectors);
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() never errors

diff --git a/src/webgpu/shader/validation/expression/call/builtin/exp.spec.ts b/src/webgpu/shader/validation/expression/call/builtin/exp.spec.ts
@@ -55,7 +55,6 @@ const valueForType = rangeForType(
 );
 
 g.test('values')
-  .batch(125)
   .desc(
     `
 Validates that constant evaluation and override evaluation of ${builtin}() rejects invalid values