feat: add v0.1 of fightertest sync testing workflow#2921
Open
Bruno-DaSilva wants to merge 2 commits intobruno/fightertest-checksum-helper-engine-changesfrom
Open
feat: add v0.1 of fightertest sync testing workflow#2921Bruno-DaSilva wants to merge 2 commits intobruno/fightertest-checksum-helper-engine-changesfrom
Bruno-DaSilva wants to merge 2 commits intobruno/fightertest-checksum-helper-engine-changesfrom
Conversation
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 11, 2026
Bruno-DaSilva
force-pushed
the
bruno/coding-docs-3
branch
from
fbc2fb8 to
ada0cab
Compare
Bruno-DaSilva
force-pushed
the
bruno/fightertest-synctest
branch
from
406efe5 to
1c5634e
Compare
This was referenced Apr 11, 2026
Bruno-DaSilva
force-pushed
the
bruno/coding-docs-3
branch
from
ada0cab to
7a4ce40
Compare
Bruno-DaSilva
force-pushed
the
bruno/fightertest-synctest
branch
from
1c5634e to
347360c
Compare
Bruno-DaSilva
changed the base branch from
bruno/coding-docs-3
to
graphite-base/2921
Bruno-DaSilva
force-pushed
the
bruno/fightertest-synctest
branch
from
347360c to
479f08e
Compare
Bruno-DaSilva
changed the base branch from
graphite-base/2921
to
bruno/fightertest-checksum-helper-engine-changes
# Purpose We're working towards multi-platform sync testing, to detect when there's sync divergences. For now this change just focuses on linux and doesn't even have a functional sync-hash extraction mechanism, but this will at least get the scaffolding of the workflow in so it can be iterated on.
Bruno-DaSilva
force-pushed
the
bruno/fightertest-synctest
branch
from
479f08e to
627e5b9
Compare
Draft
3 tasks
…rkflow BAR's fightertest Lua code now produces fightertest_synchash.txt with per-frame checksums. Uncomment the hash assertion step and include the file in uploaded artifacts. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.
Purpose
We're working towards multi-platform sync testing, to detect when there's sync divergences.
For now this change just focuses on linux and doesn't even have a functional sync-hash extraction mechanism, but this will at least get the scaffolding of the workflow in so it can be iterated on.
Part 1 of #2910
@p2004a let me know if you have any thoughts now that this is a bit more concrete. I'll continue adding on in the stack so you can see how I intend to build on this.
AI Disclosure: claude code + me :^)