MagicBeanDev/Hookshot

Archived

Watch

Fork 0

docs(tested): record paired Linux RTX 4090 rolling results #2778

Open

github-sync wants to merge 2 commits from github/pr/2778 into main

github-sync commented

2026-11-12 12:37:08 +00:00

Owner

@mooki000 opened this pull request on GitHub · 2026-11-12 12:34 UTC · original

What this is for: keep the Linux RTX 4090 results from #2595 in the tested-configurations table. What is measured: twelve rows covering four builds and three scene/settings pairs, with both repeats' log-window FPS medians and ranges. Not measured: other cards, Windows or presentation through a real window.

What changed

Adds v2.1.0, pre-fix main ef81aee7, main 46780503, and this morning's main 51da0464, newest build first. The newest rows also retain their contemporaneous 46780503 controls rather than treating the overnight figures as that comparison.
Folds repeats and unchanged cells as the existing table does. Logged version strings retain the diagnostic-only instrumentation; 240 fps is explicitly a cap, not Uncapped.

How they were measured

Arch Linux, i9-13900K, RTX 4090, NVIDIA 615.78.08; 2048x1152, warmed caches. Timed camera sweeps in Kokiri Forest and Hyrule Field used alternating runs in exclusive timing holds; every retained run met the quiet criteria (other CPU below three cores on average and six at the peak).

These are medians and ranges of complete 120-present game-log windows, not the trace-derived whole-window means in the overnight comparison and morning comparison. Extraction checks retained each quoted source line and checked build, device, settings, quiet eligibility and fault counts. Existing captures show the loaded scenes; camera views are not pixel-locked.

Documentation-only: git diff --check passes. No dedicated documentation checker or test reading this table was found, so there was no matching test to run through tests/run_all.py --strict --jobs 1. No code or fixture changed and no new game run was needed.

Limits

One card, no window, a timed route—not scanout, physical-input testing, continuous flicker proof or a long-session memory test. Each row carries the date its runs ended. The fixed-release reference was one overnight session and was not rerun beside every newer build: its Kokiri Forest runs ended before midnight (2026-11-11), its Hyrule Field and DLAA runs after it (2026-11-12), and those two rows say so.

Assisted-by: GPT-6 Astra
Co-Authored-By: GPT-6 Astra [email protected]

> [@mooki000](https://github.com/mooki000) opened this pull request on GitHub · 2026-11-12 12:34 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2778) **What this is for:** keep the Linux RTX 4090 results from #2595 in the tested-configurations table. **What is measured:** twelve rows covering four builds and three scene/settings pairs, with both repeats' log-window FPS medians and ranges. **Not measured:** other cards, Windows or presentation through a real window. ## What changed - Adds v2.1.0, pre-fix main `ef81aee7`, main `46780503`, and this morning's main `51da0464`, newest build first. The newest rows also retain their contemporaneous `46780503` controls rather than treating the overnight figures as that comparison. - Folds repeats and unchanged cells as the existing table does. Logged version strings retain the diagnostic-only instrumentation; 240 fps is explicitly a cap, not Uncapped. ## How they were measured Arch Linux, i9-13900K, RTX 4090, NVIDIA 615.78.08; 2048x1152, warmed caches. Timed camera sweeps in Kokiri Forest and Hyrule Field used alternating runs in exclusive timing holds; every retained run met the quiet criteria (other CPU below three cores on average and six at the peak). These are medians and ranges of complete 120-present **game-log windows**, not the trace-derived whole-window means in the [overnight comparison](https://git.magicbean.dev/MagicBeanDev/Hookshot/issues/2595) and [morning comparison](https://git.magicbean.dev/MagicBeanDev/Hookshot/issues/2595). Extraction checks retained each quoted source line and checked build, device, settings, quiet eligibility and fault counts. Existing captures show the loaded scenes; camera views are not pixel-locked. Documentation-only: `git diff --check` passes. No dedicated documentation checker or test reading this table was found, so there was no matching test to run through `tests/run_all.py --strict --jobs 1`. No code or fixture changed and no new game run was needed. ## Limits One card, no window, a timed route—not scanout, physical-input testing, continuous flicker proof or a long-session memory test. Each row carries the date its runs ended. The fixed-release reference was one overnight session and was not rerun beside every newer build: its Kokiri Forest runs ended before midnight (2026-11-11), its Hyrule Field and DLAA runs after it (2026-11-12), and those two rows say so. Assisted-by: GPT-6 Astra Co-Authored-By: GPT-6 Astra <[email protected]>

github-sync added 1 commit

2026-11-12 12:37:08 +00:00

docs(tested): record paired Linux RTX 4090 rolling results 8c2aa12487

Keep both repeats of each log-window measurement, distinguish instrumented builds and retain the contemporaneous morning controls. Refs #2595.

Assisted-by: GPT-6 Astra
Co-Authored-By: GPT-6 Astra <[email protected]>

github-sync added 1 commit

2026-11-12 14:41:10 +00:00

docs(tested): say why two release rows carry the later date; name the kernel 5bc519f236

The v2.1.0 reference runs were one overnight session: Kokiri Forest ended
before midnight (2026-11-11), Hyrule Field and the DLAA scene after it
(2026-11-12). The dates were right but read like a re-run; the two rows
now say so. The OS cell reads "Arch Linux (kernel ...)".

Assisted-by: Claude Fable 5.2
Co-Authored-By: Claude Fable 5.2 <[email protected]>

github-sync referenced this pull request

2026-11-12 14:44:35 +00:00

docs: tested configurations: RX 5600 XT 6 GB (RDNA1, no mesh shaders), Windows 11, main de866017 #2715

github-sync referenced this pull request

2026-11-12 14:47:49 +00:00

perf(builder): open the files of the damaged-texture check several at a time #2769

github-sync referenced this pull request

2026-11-12 14:47:51 +00:00

docs: add tested-configurations rows for DLSS Frame Generation on an RTX 4090 (Windows) #2771

github-sync commented

2026-11-12 14:48:11 +00:00

Author

Owner

@Joey-Magnus reviewed on GitHub · 2026-11-12 14:35 UTC · original

These are docs only, and I checked them against the description. The four builds, three scene/settings pairs and paired controls are laid out clearly, and saying that these are log-window medians, not the trace-derived means on #2595, is useful. One inconsistency: the v2.1.0 block starts with a 2026-11-11 row (docs/tested-configurations.md:26), but its Hyrule Field and DLAA continuation rows directly below it are dated 2026-11-12 with "as above" everywhere else. The description says the fixed-release measurements keep their earlier date, so those two rows probably want 2026-11-11 too. Otherwise a reader takes them as a re-run. Also, the OS cell "Arch Linux 7.2.15-1-omarchy" reads like a kernel version, so "Arch Linux (kernel 7.2.15-1-omarchy)" would be clearer. This conflicts with #2769 and #2771 on the first table row, so whichever lands later needs a one-line rebase.

> [@Joey-Magnus](https://github.com/Joey-Magnus) reviewed on GitHub · 2026-11-12 14:35 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2778#pullrequestreview-5876254032) These are docs only, and I checked them against the description. The four builds, three scene/settings pairs and paired controls are laid out clearly, and saying that these are log-window medians, not the trace-derived means on #2595, is useful. One inconsistency: the v2.1.0 block starts with a 2026-11-11 row (docs/tested-configurations.md:26), but its Hyrule Field and DLAA continuation rows directly below it are dated 2026-11-12 with "as above" everywhere else. The description says the fixed-release measurements keep their earlier date, so those two rows probably want 2026-11-11 too. Otherwise a reader takes them as a re-run. Also, the OS cell "Arch Linux 7.2.15-1-omarchy" reads like a kernel version, so "Arch Linux (kernel 7.2.15-1-omarchy)" would be clearer. This conflicts with #2769 and #2771 on the first table row, so whichever lands later needs a one-line rebase.

github-sync commented

2026-11-12 14:48:11 +00:00

Author

Owner

@mooki000 commented on GitHub · 2026-11-12 14:38 UTC · original

Thanks. Both points taken in 5bc519f2:

The dates: they are the dates the runs ended, and the release reference straddled midnight. Its Kokiri Forest runs ended 23:29 and 23:32 on 2026-11-11; its Hyrule Field runs 02:01 and 02:04 and its DLAA runs 02:12 and 02:14 on 2026-11-12 (local time, from the runs' own records). So I kept the dates and made the two rows say it: "from the same overnight session as the Kokiri Forest row above, measured after midnight (hence the later date; not a re-run)". The description's sentence is corrected the same way.
The OS cell now reads "Arch Linux (kernel 7.2.15-1-omarchy)".

On #2769 and #2771: understood, I will rebase the first row if either lands first.

> [@mooki000](https://github.com/mooki000) commented on GitHub · 2026-11-12 14:38 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2778#issuecomment-6826600044) Thanks. Both points taken in `5bc519f2`: - **The dates:** they are the dates the runs ended, and the release reference straddled midnight. Its Kokiri Forest runs ended 23:29 and 23:32 on 2026-11-11; its Hyrule Field runs 02:01 and 02:04 and its DLAA runs 02:12 and 02:14 on 2026-11-12 (local time, from the runs' own records). So I kept the dates and made the two rows say it: "from the same overnight session as the Kokiri Forest row above, measured after midnight (hence the later date; not a re-run)". The description's sentence is corrected the same way. - **The OS cell** now reads "Arch Linux (kernel 7.2.15-1-omarchy)". On #2769 and #2771: understood, I will rebase the first row if either lands first.

github-sync force-pushed github/pr/2778 from 5bc519f236 to 2dcb57d8f2

2026-11-12 17:57:47 +00:00

Compare

github-sync commented

2026-11-12 17:59:15 +00:00

Author

Owner

@mooki000 commented on GitHub · 2026-11-12 17:21 UTC · original

Rebased onto current main (86a9cda2) as 2dcb57d8: today's merges had added rows at the same place in the table. Nothing else changed: the twelve added rows are byte-identical to the ones reviewed, they stay together in one block (so each "as above" still points at the row it did), and every row main gained is kept in main's order.

> [@mooki000](https://github.com/mooki000) commented on GitHub · 2026-11-12 17:21 UTC · [original](https://github.com/NotRexed/Hookshot/pull/2778#issuecomment-6830761609) Rebased onto current main (`86a9cda2`) as `2dcb57d8`: today's merges had added rows at the same place in the table. Nothing else changed: the twelve added rows are byte-identical to the ones reviewed, they stay together in one block (so each "as above" still points at the row it did), and every row main gained is kept in main's order.

github-sync referenced this pull request

2026-11-12 19:00:48 +00:00

fix(tests): the write-back shadow remake check builds on Windows #2868

github-sync force-pushed github/pr/2778 from 2dcb57d8f2 to b2f2fecccf

2026-11-12 19:00:57 +00:00

Compare

Commenting is not possible because the repository is archived.

No reviewers