Skip to content

chore: PairingProverExt is vacuous#1907

Merged
jonathanpwang merged 1 commit into
feat/new-executionfrom
chore/pairing-general
Jul 26, 2025
Merged

chore: PairingProverExt is vacuous#1907
jonathanpwang merged 1 commit into
feat/new-executionfrom
chore/pairing-general

Conversation

@jonathanpwang

@jonathanpwang jonathanpwang commented Jul 26, 2025

Copy link
Copy Markdown
Contributor

PairingProverExt is backend agnostic because it is vacuous and the pairing extension only consists of phantom sub-executors.

Also changed a trait bound from PrimeField32 to just Field

`PairingProverExt` is backend agnostic because it is vacuous and the
pairing extension only consists of phantom sub-executors.
@github-actions

Copy link
Copy Markdown
group app.proof_time_ms app.cycles app.cells_used leaf.proof_time_ms leaf.cycles leaf.cells_used
verify_fibair (-60 [-5.3%]) 1,074 322,700 17,672,408 - - -
fibonacci (-29 [-1.3%]) 2,219 3,000,556 51,508,517 - - -
regex (+32 [+0.4%]) 7,523 8,330,866 165,431,684 - - -
ecrecover (-16 [-1.4%]) 1,145 274,568 8,596,011 - - -
pairing (-51 [-1.4%]) 3,647 3,725,930 93,105,233 - - -

Commit: f413849

Benchmark Workflow

@codspeed-hq

codspeed-hq Bot commented Jul 26, 2025

Copy link
Copy Markdown

CodSpeed WallTime Performance Report

Merging #1907 will not alter performance

Comparing chore/pairing-general (f413849) with feat/new-execution (a62e015)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 24 untouched benchmarks

@jonathanpwang jonathanpwang merged commit d572c40 into feat/new-execution Jul 26, 2025
31 checks passed
@jonathanpwang jonathanpwang deleted the chore/pairing-general branch July 26, 2025 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant