From 88a1887cc28598043e97832b3cd154d217faf832 Mon Sep 17 00:00:00 2001 From: Eric Engestrom Date: Tue, 10 Feb 2026 16:22:53 +0100 Subject: [PATCH] Revert "add VK CTS validation report for a0 interpolation fix" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit 4eeda739c4ea856e37b9daced4564a17e30aceaa. Looks like @ajax has been playing with claude "ai" and accidentally committed and pushed things to main. 1. That doesn't belong here. 2. We shouldn't bypass MR review unless in emergency situations. This seems like a good time to remind people to not have a push-able upstream remote, to avoid an accidental `git push` or something else doing that. One way to do this is to change the push url like this, assuming your upstream remote is called `origin` (default if you didn't pick something else): git remote set-url --push origin invalid-url `invalid-url` will fail if you try to push to it, catching your mistakes :) Spotted-by: Maíra Canal Part-of: --- .gitignore | 6 --- claude-nwnk/a0-interpolation-cts-report.md | 58 ---------------------- 2 files changed, 64 deletions(-) delete mode 100644 claude-nwnk/a0-interpolation-cts-report.md diff --git a/.gitignore b/.gitignore index 72333560e01..1ccc49fc360 100644 --- a/.gitignore +++ b/.gitignore @@ -7,9 +7,3 @@ /build .venv/ *.mda.tar - -# Gas Town (added by gt) -.runtime/ -.claude/ -.beads/ -.logs/ diff --git a/claude-nwnk/a0-interpolation-cts-report.md b/claude-nwnk/a0-interpolation-cts-report.md deleted file mode 100644 index a665729fae7..00000000000 --- a/claude-nwnk/a0-interpolation-cts-report.md +++ /dev/null @@ -1,58 +0,0 @@ -# VK CTS validation: fury's a0 interpolation fix - -convoy: hq-cv-wr4iu (llvmpipe-rasterizer-validation) - -## summary - -fury's fix changes interpolation origin from framebuffer (0,0) to vertex 0 -position for numerical stability with triangles far from origin. - -**result: no regressions, no fixes detected. fix is safe from a CTS perspective.** - -## test results - -tested dEQP-VK categories focused on rasterization/interpolation. - -### dEQP-VK.rasterization.* - -| build | passed | failed | not supported | -|-------|--------|--------|---------------| -| baseline | 7448 | 0 | 7631 | -| with fix | 7448 | 0 | 7631 | - -### dEQP-VK.draw.* - -| build | passed | failed | not supported | -|-------|--------|--------|---------------| -| baseline | 24015 | 0 | 6729 | -| with fix | 24015 | 0 | 6729 | - -### dEQP-VK.pipeline.monolithic.multisample.*.verify_interpolation.* - -| build | passed | failed | not supported | -|-------|--------|--------|---------------| -| baseline | 15 | 0 | 18 | -| with fix | 15 | 0 | 18 | - -## notes - -- the fix is designed for numerical stability with triangles far from origin, - which the CTS may not explicitly stress. -- identical results baseline vs fix suggests no functional regression. -- the "not supported" counts are expected - lavapipe doesn't expose all - features (shader_tile_image, android external formats, high sample counts, etc). - -## files changed - -12 files in src/gallium/drivers/llvmpipe/: -- lp_bld_interp.c -- lp_linear.c -- lp_linear_fastpath.c -- lp_linear_interp.c -- lp_linear_priv.h -- lp_linear_sampler.c -- lp_rast.c -- lp_setup_line.c -- lp_setup_point.c -- lp_state_fs_linear.c -- lp_state_setup.c