Repository navigation
feat: the proof timing report from the browser, on every pull request - #20
Closed
sergeytimoshin wants to merge 1 commit into
Closed
sergeytimoshin wants to merge 1 commit into
sergeytimoshin wants to merge 1 commit into
Conversation
Proof timing: chromeAfter: this pull request at Medians of one browser, by shape (notes → outputs). Runner numbers are for comparing commits, not for how fast a device is.
Failed:
|
sergeytimoshin
force-pushed
the
sergei/proof-timing
branch
2 times, most recently
from
October 16, 2026 18:29
98cd74f to
ccbfc20
Compare
`examples/browser/scripts/proof-timing.mjs` drives the benchmark page in headless Chrome through the four scenarios the mobile SDK reports (first run including the key's fetch and decode, warm, first run after a reload with the key in the raw-key cache, repeated) for transfers of 1 to 24 notes and a merge of 24, and prints one `ZOLANA_TIMING` line per proof; `scripts/proof-timing-report.mjs` renders the table before and after a change. A bench job beside the checks comments the report on every pull request against the last commit of main, and after each push to main the report joins the history on the proof-timing branch.
sergeytimoshin
force-pushed
the
sergei/proof-timing
branch
from
October 16, 2026 21:36
ccbfc20 to
049c300
Compare
Collaborator
Author
|
Superseded: this change reached |
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
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.
The browser prover gets the proof timing report the mobile SDK has: four scenarios for every shape, before and after a change, commented on every pull request, with a history of
main.examples/browser/scripts/proof-timing.mjs: drives the benchmark page in headless Chrome (selenium, astest:browser) through first run including the key's fetch and decode, warm, first run after a page reload (raw-key cache) and repeated, for transfers of 1 to 24 notes and a merge of 24 (ZOLANA_TIMING_SHAPESnarrows it); a failed or stalled shape prints anerrorline and the others run in a new browser.scripts/proof-timing-report.mjs(+ test): the table withbefore/afterrows, the download bracket, the stage split and failed shapes;proof-timing-ci.shtakes the baseline from the history and writes the job summary and one PR comment;proof-timing-history.{mjs,sh}append eachmainreport to theproof-timingbranch and rewrite its README.wasm/prover-wasm/testdata:transfer-{1,2,4,8,16,24}x2.jsonandmerge-24x1.json, valid requests on trees of their own (the mobile SDK'stools/prove-fixtureswrote them), with Go tests of their shapes;build-wasm.shstages the transfer requests.ci.yml: abenchjob besidecheck, cancelled by a newer push to the PR, and aproof-timing-historyjob withcontents: writeafter each push tomain.On this branch's runtime (0.1.0) 24x2 crashes the browser and the merge stalls at the kernel's 1 GiB memory cap, so the report shows both failed until #21 and #22 bring runtime 0.2.0. The
proof-timingbranch appears with the first push tomainafter merge; until then the PR comment has nobeforerows.