build(release): the RTX 5090 fork list's note says what its build changed (#2143 follow-up) #2703

Open
github-sync wants to merge 1 commit from github/pr/2703 into main
Owner

@Brett-D opened this pull request on GitHub · 2026-11-12 07:28 UTC · original

Follow-up to #2143, which merged before I could answer Joey-Magnus's and hckrman101's question about brett-d-rtx-5090-fork-fg-2x. This changes only that list's note, to say what its build was.

The "fork" was local builds of main at 0828863 (9 Nov), plus changes that don't touch what goes into a record: DLSS-NR and an FXAA option in the window-scaling pass, a sampler cache and its descriptor-reuse check, a texture budget, menu rows and a voice feature. There are no changes to shader translation, program variants, target formats or draw state.

python -I nx2/pipeline_states.py check passes. The list itself is unchanged.

Refs #2034, #2143

> [@Brett-D](https://github.com/Brett-D) opened this pull request on GitHub · 2026-11-12 07:28 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2703) Follow-up to #2143, which merged before I could answer Joey-Magnus's and hckrman101's question about `brett-d-rtx-5090-fork-fg-2x`. This changes only that list's note, to say what its build was. The "fork" was local builds of main at 0828863 (9 Nov), plus changes that don't touch what goes into a record: DLSS-NR and an FXAA option in the window-scaling pass, a sampler cache and its descriptor-reuse check, a texture budget, menu rows and a voice feature. There are no changes to shader translation, program variants, target formats or draw state. `python -I nx2/pipeline_states.py check` passes. The list itself is unchanged. Refs #2034, #2143 <!-- github-sync:item:2703 -->
Author
Owner

@Joey-Magnus approved on GitHub · 2026-11-12 14:34 UTC · original

Thanks, this answers the question I asked on #2143. The list was recorded on local builds of main at 08288630 (9 Nov, which I can see on main), and the local changes are host-side only: DLSS-NR, FXAA in the window-scaling pass, the sampler/descriptor caching, the texture budget, menu and voice. None of those touches shader translation, program variants, target formats or draw state, so the records describe pipelines main makes. Records from the older-main sessions can at worst name pipelines that are never hit, which costs nothing. I merged this into current main and ran python -I nx2/pipeline_states.py check: it passes, and only the note changed. The one failing job, Python source checks part 1/2, is test_release_scripts.py's test_nightly_two_runs_at_once_end_at_the_newer_commit. That test is flaky and unrelated to this file, and a re-run should clear it.

> [@Joey-Magnus](https://github.com/Joey-Magnus) approved on GitHub · 2026-11-12 14:34 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2703#pullrequestreview-5876235610) Thanks, this answers the question I asked on #2143. The list was recorded on local builds of main at 08288630 (9 Nov, which I can see on main), and the local changes are host-side only: DLSS-NR, FXAA in the window-scaling pass, the sampler/descriptor caching, the texture budget, menu and voice. None of those touches shader translation, program variants, target formats or draw state, so the records describe pipelines main makes. Records from the older-main sessions can at worst name pipelines that are never hit, which costs nothing. I merged this into current main and ran `python -I nx2/pipeline_states.py check`: it passes, and only the note changed. The one failing job, Python source checks part 1/2, is `test_release_scripts.py`'s `test_nightly_two_runs_at_once_end_at_the_newer_commit`. That test is flaky and unrelated to this file, and a re-run should clear it. <!-- github-sync:r:5876235610 -->
Commenting is not possible because the repository is archived.
No description provided.