Prune the render configuration space (promote / remove non-default paths) #1098

New issue

Open

opened 2026-10-26 02:25:11 +00:00 by sky_line · 0 comments

sky_line commented

2026-10-26 02:25:11 +00:00

Owner

Steps 1-2 are in progress separately (A/B and debug settings Persist=0 + a "non-default render settings" log line and bug-report field; code-default literals aligned with settings.xml).

Remaining, as small approved batches:
3. Remove the #640 forward fallback under DepthPrepass.
4. Remove the "old rule" family: ReSTIR soft cull / ConvMultiLight / LightCut, ShadowSunOneRay, RTShadowHint, GpuSpinMovingLinksets, RigWeldMode 0, PointShadowMaps, LightStage, OcclusionCull / Occlusion2Phase / SkyLUT.
5. Remove the in-viewer register-probe variants once scripts/perf/regprobe covers the draw shaders (and shader-clock twins if superseded).
6. Remove RenderVKShadowOpaqueFirst (no win; the AMD cull-mask variant would be new work under #1085).
7. Promote the shipped lighting splits (DeferredFeatureSplit, DeferClassifyFused, froxel-only LightCulling, GbufSlimVS / GbufTerrainSplit, DepthOnlyVS, TonemapLut, ProbeDeferred / ProbeCaptureSlim, RTReflectionsRayList, one ProbeFormat), one commit each.
8. Remove off-by-default features that are not pulling their weight.
9. RTShadowMerge chosen automatically per vendor (merged path is ~4x worse on AMD RDNA4); drop the Preferences toggle.
10. Sweep the ~100 default-1 "0 = old behaviour" kill switches per subsystem.
11. Fused kernels last: they are also the allocation-failure / still-compiling fallback, so they need a skip/retry/degrade replacement first. #1039 queue bits stay (non-persistent) until the M6 decision.

Standing rule (owner, 2026-10-25): keep the configuration space minimal; non-default paths earn their place or are removed; experiment toggles never persist. Audit: CONFIG_PRUNE_AUDIT.md in the team research repo (985 RenderVK* settings, 922 persisted; ~73 of ~400 shader variants used by no default path). Steps 1-2 are in progress separately (A/B and debug settings Persist=0 + a "non-default render settings" log line and bug-report field; code-default literals aligned with settings.xml). Remaining, as small approved batches: 3. Remove the #640 forward fallback under DepthPrepass. 4. Remove the "old rule" family: ReSTIR soft cull / ConvMultiLight / LightCut, ShadowSunOneRay, RTShadowHint, GpuSpinMovingLinksets, RigWeldMode 0, PointShadowMaps, LightStage, OcclusionCull / Occlusion2Phase / SkyLUT. 5. Remove the in-viewer register-probe variants once scripts/perf/regprobe covers the draw shaders (and shader-clock twins if superseded). 6. Remove RenderVKShadowOpaqueFirst (no win; the AMD cull-mask variant would be new work under #1085). 7. Promote the shipped lighting splits (DeferredFeatureSplit, DeferClassifyFused, froxel-only LightCulling, GbufSlimVS / GbufTerrainSplit, DepthOnlyVS, TonemapLut, ProbeDeferred / ProbeCaptureSlim, RTReflectionsRayList, one ProbeFormat), one commit each. 8. Remove off-by-default features that are not pulling their weight. 9. RTShadowMerge chosen automatically per vendor (merged path is ~4x worse on AMD RDNA4); drop the Preferences toggle. 10. Sweep the ~100 default-1 "0 = old behaviour" kill switches per subsystem. 11. Fused kernels last: they are also the allocation-failure / still-compiling fallback, so they need a skip/retry/degrade replacement first. #1039 queue bits stay (non-persistent) until the M6 decision.

sky_line added the

rendering

perf

labels

2026-10-26 02:25:11 +00:00

sky_line added this to the Slipstream project

2026-10-26 02:25:11 +00:00

sky_line self-assigned this

2026-10-26 02:58:28 +00:00