docs(tested): retain historical Hookshot Linux rolling results #53
No reviewers
Labels
No labels
accessibility
autorelease: pending
autorelease: tagged
bug
dependencies
documentation
duplicate
enhancement
github_actions
good first issue
help wanted
invalid
needs measurements
question
testing wanted
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
MagicBeanDev/Longshot!53
Loading…
Reference in a new issue
No description provided.
Delete branch "mooki/Longshot:docs/tested-configurations-linux-rolling"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Why
Keep the paired Linux RTX 4090 results recorded before the project was renamed, so a future regression can still be traced to its actual build, driver and scene. These are historical results of builds named Hookshot, not fresh Longshot frame-rate measurements: the history is useful precisely because its identities and limitations are retained.
What changed
8ef84611, pre-fix mainef81aee7, main46780503, and main51da0464, across three scene/settings pairs. They retain 24 primary runs plus the six contemporaneous morning controls.docs/tested-configurations.mdchanges in the repository.The historical measurements used Arch Linux, i9-13900K, RTX 4090 / NVIDIA 615.78.08, 2048×1152, warmed caches and timed camera sweeps in Kokiri Forest and Hyrule Field. The two main builds alternated in each authorized quiet timing hold; the release reference came from separate earlier holds. The rows report medians/ranges of complete 120-present game-log windows, not the trace-derived means in the overnight and morning reports. A 240 fps cap is not Uncapped. None of those timing runs was repeated for this move.
These rows are the Linux companion record that issue 69's “Hookshot continuity” section points to on GitHub.
Flags and settings
None. This changes documentation rows only; the recorded settings and existing test-command arguments are not new or changed controls.
Measurements
Branch state: on main
f41db065cf6667ec638a10aa6990099392d10dca; headc946fae6b5755c89cb793227d32ec7643aec5707; whole game-free suite on this head:509 passed, 0 failed, 19 skipped(mainf41db065cf6667ec638a10aa6990099392d10dca:509 passed, 0 failed, 19 skipped); the commits are unchanged by rebases apart from the results table. The updater test that failed locally on older mains left main together with the in-game updater.main:
d4a69456c9(2026-11-12)this PR: docs/tested-configurations-linux-rolling @
a17856e099Original measurement record: checked at
a17856e0990cb1e38616e2e832220da0501fecbcwith the historical labels/links applied, followed only by the documentation receipt rows. No new rebase is made for this description update; the publication base is refreshed centrally when the PR is marked ready. The measured table rows and their historical identities are unchanged.Hardware: Core i9-13900K, 94 GiB RAM, Arch Linux kernel 7.2.15-1-omarchy, Python 3.12.16; full-suite default compiler Clang 23.1.3. No GPU used for these fresh checks.
Scene: game-free table/source checks; no game, save or rendering settings apply.
Method: unmeasured provenance-check warm-up, then before/after/before/after with fresh HOME directories. The retained check recomputes each stored median, range and window count from its line-numbered log excerpts, then checks the table's twelve contiguous rows, all 30 runs, original build fingerprints, dates and contemporaneous controls. Main has no such block, so the before result is an expected missing-documentation failure, not a runtime defect. The whole suite uses
--jobs 6 --strict. The base suite was run once for all moved pull requests of this day, on this commit; the branch suite was run once. Suite durations are not speed measurements.How to reproduce: run
python -I -B tests/run_all.py --jobs 6 --stricton each named source tree. For the documentation result, compare the twelve contiguous added rows with the original log-window excerpts and reports linked above, retaining both repeats, the six contemporaneous controls and the explicit after-midnight dates; no new game run is implied by that comparison.d4a69456a17856e0Actual documentation-check output, identical in both repetitions:
Actual whole-suite output:
Both fail
tests/test_updater_asset_digest.pywith this inherited local diagnostic (temporary path omitted):The shared main suite also failed two timing-sensitive assertions in
tests/test_appimage_package_retry.pyunder heavy load: the enrollment retry sequence and124 != 0. Its separate isolated follow-up passed all nine cases, without erasing the original failure. This branch passes that check in its whole suite. The 20 skips are explicit platform/game/GPU exclusions. The suite is not described as all green.Testing
Tested: the whole game-free source suite on the exact head in Branch state; the original paired documentation checks recomputed all 30 runs' log-window medians/ranges/counts and checked the twelve-row grouping, dates, build fingerprints and controls. All six row links resolved when checked, both cited GitHub comments were read, and the three historical pictures were inspected. These are correctness/provenance checks, not fresh speed measurements.
Not tested: No new Longshot game build/run, fresh frame-rate comparison, Windows run, other GPU, scanout, physical input, continuous-flicker proof or long-session memory test. Documentation cannot establish current Longshot performance. The release reference was not paired with every newer build: its forest runs ended on 2026-11-11 and its field/DLAA runs after midnight on 2026-11-12; the rows state that, rather than pretending they were reruns.
Needs other testers:
None for this documentation-rows-only change: it changes no code, fixture, setting or hardware-dependent behavior.
Screenshots
Nothing on screen changes in this PR. These inspected historical captures show the scenes behind the retained Hookshot
51da0464measurements, with diagnostic-only buildf6cfc513f/ source1519af1c, on 2026-11-12. They are not pixel-locked before/after views or new Longshot evidence.Risk and rollback
A reader could mistake dated Hookshot results for current Longshot measurements, or a later table merge could break the contiguous “as above” grouping. Keep the explicit historical labels, original commits and complete row block. Reverting this documentation-only change removes the added rows without affecting the game or saves.
Reviewer notes
Review the four build identities, six contemporaneous morning controls and after-midnight release dates first. The row values are game-log-window statistics, deliberately different from the linked trace means; no historical run is relabelled as a fresh Longshot benchmark. The uploaded pictures and their captions are unchanged.
Issues
Original rolling-results report: https://github.com/NotRexed/Hookshot/issues/2595.
See #69
Original documentation PR: https://github.com/NotRexed/Hookshot/issues/2778. The earlier public all-issues check on 2026-11-12 found no corresponding rolling-results issue; the forge now has its own continuation at #69. Issue #45 of this forge concerns table merge conflicts, not these measurements; no issue is closed by this move.
Checklist
For everyone
type(scope): description).main.python -I -B tests/run_all.pyrun and passing (any failure that also fails onmainis named above).third_party/untouched (a new folder with aPROVENANCE.mdinstead).If applicable
Assisted-by:/Co-Authored-By:trailers.docs/tested-configurations.mdfor each measured run.Moved from NotRexed/Hookshot#2778.
Assisted-by: GPT-6 Astra
Co-Authored-By: GPT-6 Astra [email protected]
Discussion carried over from NotRexed/Hookshot#2778
Every comment that was readable on the original thread when it was copied on 2026-11-12, in order, unchanged except that issue and pull request numbers are written as links to the old repository and names carry no @. Comments by an account that has since been closed there could no longer be read and are missing; where a reply quotes them, the quote remains.
Joey-Magnus, 2026-11-12 14:35 UTC (review: comment, 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 NotRexed/Hookshot#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 NotRexed/Hookshot#2769 and NotRexed/Hookshot#2771 on the first table row, so whichever lands later needs a one-line rebase.
mooki000, 2026-11-12 14:38 UTC (comment, original):
Thanks. Both points taken in
5bc519f2:On NotRexed/Hookshot#2769 and NotRexed/Hookshot#2771: understood, I will rebase the first row if either lands first.
mooki000, 2026-11-12 17:21 UTC (comment, original):
Rebased onto current main (
86a9cda2) as2dcb57d8: 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.docs(tested): retain historical Hookshot Linux rolling resultsto WIP: docs(tested): retain historical Hookshot Linux rolling resultsBack to work in progress, for its description only. This was opened shortly before #48 merged and follows the earlier template, so it does not pass the new description check (no Flags and settings and no Risk and rollback section, table columns named Before and After). The code and the measurements are unchanged: the whole suite passes on this exact commit apart from one local .NET failure that also fails on main. The description is being moved to the new template and the branch rebased onto
d725b363; it comes back throughpr readyonce the check passes.Updated the description to the current template and kept all three uploaded pictures, the historical results and the original measurement identities. Added the link to the rolling-results issue (#69): these are its historical Linux companion rows. Rebased onto main
f41db065ckeeping both sides' rows of the results table; whole suite on the new headc946fae6b: 509 passed, 0 failed, 19 skipped. Ready for review.WIP: docs(tested): retain historical Hookshot Linux rolling resultsto docs(tested): retain historical Hookshot Linux rolling results4ba4554baatoc946fae6b5View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.