From 6e719da3033f94d94f291ce58918948521443e01 Mon Sep 17 00:00:00 2001 From: Gregg Tavares Date: Fri, 10 Jan 2025 14:36:57 -0800 Subject: [PATCH] Remove api/operation/texture_view/read.spec.ts These seem to be covered by the texture builtin tests and they are unimplemented. --- .../api/operation/texture_view/read.spec.ts | 56 ------------------- src/webgpu/listing_meta.json | 3 - 2 files changed, 59 deletions(-) delete mode 100644 src/webgpu/api/operation/texture_view/read.spec.ts diff --git a/src/webgpu/api/operation/texture_view/read.spec.ts b/src/webgpu/api/operation/texture_view/read.spec.ts deleted file mode 100644 index ce2e5055a8f2..000000000000 --- a/src/webgpu/api/operation/texture_view/read.spec.ts +++ /dev/null @@ -1,56 +0,0 @@ -export const description = ` -Test the result of reading textures through texture views with various options. - -All x= every possible view read method: { - - {unfiltered, filtered (if valid), comparison (if valid)} sampling - - storage read {vertex, fragment, compute} - - no-op render pass that loads and then stores - - depth comparison - - stencil comparison -} - -Format reinterpretation is not tested here. It is in format_reinterpretation.spec.ts. - -TODO: Write helper for this if not already available (see resource_init, buffer_sync_test for related code). -`; - -import { makeTestGroup } from '../../../../common/framework/test_group.js'; -import { GPUTest } from '../../../gpu_test.js'; - -export const g = makeTestGroup(GPUTest); - -g.test('format') - .desc( - `Views of every allowed format. - -- x= every texture format -- x= sampleCount {1, 4} if valid -- x= every possible view read method (see above) -` - ) - .unimplemented(); - -g.test('dimension') - .desc( - `Views of every allowed dimension. - -- x= a representative subset of formats -- x= {every texture dimension} x {every valid view dimension} - (per gpuweb#79 no dimension-count reinterpretations, like 2d-array <-> 3d, are possible) -- x= sampleCount {1, 4} if valid -- x= every possible view read method (see above) -` - ) - .unimplemented(); - -g.test('aspect') - .desc( - `Views of every allowed aspect of depth/stencil textures. - -- x= every depth/stencil format -- x= {"all", "stencil-only", "depth-only"} where valid for the format -- x= sampleCount {1, 4} if valid -- x= every possible view read method (see above) -` - ) - .unimplemented(); diff --git a/src/webgpu/listing_meta.json b/src/webgpu/listing_meta.json index 6b8ced65d043..a44b8fc0790e 100644 --- a/src/webgpu/listing_meta.json +++ b/src/webgpu/listing_meta.json @@ -210,9 +210,6 @@ "webgpu:api,operation,storage_texture,read_write:basic:*": { "subcaseMS": 5.000 }, "webgpu:api,operation,texture_view,format_reinterpretation:render_and_resolve_attachment:*": { "subcaseMS": 14.488 }, "webgpu:api,operation,texture_view,format_reinterpretation:texture_binding:*": { "subcaseMS": 17.225 }, - "webgpu:api,operation,texture_view,read:aspect:*": { "subcaseMS": 0.601 }, - "webgpu:api,operation,texture_view,read:dimension:*": { "subcaseMS": 0.701 }, - "webgpu:api,operation,texture_view,read:format:*": { "subcaseMS": 1.100 }, "webgpu:api,operation,texture_view,write:aspect:*": { "subcaseMS": 0.700 }, "webgpu:api,operation,texture_view,write:dimension:*": { "subcaseMS": 0.601 }, "webgpu:api,operation,texture_view,write:format:*": { "subcaseMS": 0.600 },