Repository navigation
chore(quest): plan an Ed25519 verify benchmark that times every library the same way - #427
Conversation
…ry the same way Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Your OpenCode Go account is out of credit. Top it up, then re-trigger Pullfrog. OpenCode Go's response
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed head 212265427b175b8a825a6702627b878ade2d918e.
No actionable findings in this planning change. The stated key-setup asymmetry matches the benchmark: rind/graviola parse inside the loop, while aws-lc-rs/dalek do so outside. The all-library goal is sound.
Implementation direction: include OpenSSL's reusable key setup and clearly label APIs that take raw key bytes. Refresh the OVH host measurements before removing their gap entries: nightly imports CPU-tier results, while host data needs a separate collection and render --from.
Verification: GitHub-only review of both changed documents, benchmark rows and report import/render paths. No benchmark implementation or results change here. I did not run quest validation or benchmarks, and the quoted performance numbers were not independently reproduced. Queue checks are skipped.
(Written by review, OpenAI)
|
quest land watch: the merge queue removed this pull request at 2122654 because it conflicts with The land fixer works on it in the background, and the watcher re-enqueues it at the back of the queue once it is fixed or judged flaky, or hands it back here with what it needs. (written by quest land watch) |

Plans
quest/m2/speed/ed25519-verify-bench.md[XS], a follow-up from moq-dev/rind#415.The
ed25519-verifybench rows aren't comparable. rind's row parses the public key inside the timed loop, about 4 µs per verify, while the aws-lc-rs and dalek rows parse it once outside. Every host's report therefore overcharges rind. With the key parsed once, rind was level with dalek or faster (25-27 µs against 29-30, on an i7-8700 with ADX off).The maintainer decided on 2026-11-07 to plan it in m2 speed.
Checks
quest checkpasses.(Written by Claude Opus 5.5)
🤖 Generated with Claude Code
Review
mechanical fix by quest land watch:
merged main at 9cd54e99b0c16d77365422d5d7ffddd7db3517ef with the quest merge driver(head efaad70)