Repository navigation
Conversation
Upstream issue incoai#190. Default off, no behavior change.
ReplayState/DeferredReplay/onVerify with static_asserts. No new .metal: verify_gdn_fused* is the existing resolve target. Upstream issue incoai#190, PR incoai#198.
linson007
changed the title
linson007
force-pushed
the
rfc/190-deferred-gdn
branch
from
October 21, 2026 18:16
a7ddcdd to
2159f78
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Commit attribution was corrected to
linson007with a metadata-only history rewrite. File trees are unchanged; historical benchmark source hashes and CI links below retain their original recorded scope.Main already implements deferred GDN replay: commit
48a94aaae6deda31bfbaec08e8a896c2d670e37afolds a decode step's retained rows into the next step. This PR now documents that mechanism against current main1482f6bbc2abf12375332725302019594150b6cb.The net diff is one documentation file. The unused
SPLASH_DEFERRED_GDN_REPLAYheader and string/Python-mirror tests are removed from the proposal. Their placeholder counted discarded suffix rows (proposed - accepted); the actual runtime records retained rows, including the anchor. They did not implement or validate the current mechanism.docs/rfc/190-deferred-gdn.mdlinks the current tape, shader, state and oracle paths and records the constraints on any further work: prior retained rows, convolution carry, tape ownership, materialization/capture/cache publication, supported layouts and both state formats. It does not introduce a second runtime flag or claim a new speedup. Related issue #190 remains the discussion point for further optimization.Merge commit
a7ddcddccontains current main without rewriting the original branch. Ahead/behind checks show zero commits behind main. Local documentation links and full-patch whitespace checks pass. There are no runtime/shader/build changes to retest locally; fresh current-head CI passes all four ordinary jobs: production/CPU runtime, every native build and sanitizers, and Python 3.12, 3.13 and 3.14 tests/source checks. The opt-in release hardware gate is skipped. The Python 3.14 job runs 1,318 tests with four skips; lint and formatting pass. Older September scaffolding CI is historical, not evidence of new runtime behavior.