Skip to content

fix: rust verifier performance#432

Open
1-leo wants to merge 3 commits intomasterfrom
fix-rust-verifier-performance
Open

fix: rust verifier performance#432
1-leo wants to merge 3 commits intomasterfrom
fix-rust-verifier-performance

Conversation

@1-leo
Copy link
Contributor

@1-leo 1-leo commented Feb 9, 2026

after some digging i found that the problem is that using k256 in wasm works but every 256 multiplication is emulated.
Switching to secp256k1 (uses the c code from bitcoin) is about 2x on native.
So i suggest we use the web bridge from @nogringo , that way we have better access to the hardware and use secp256k1 on native platforms

@1-leo 1-leo self-assigned this Feb 9, 2026
@codecov
Copy link

codecov bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.15%. Comparing base (3d4e3a5) to head (8d2759d).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #432   +/-   ##
=======================================
  Coverage   75.15%   75.15%           
=======================================
  Files         144      144           
  Lines        5755     5755           
=======================================
  Hits         4325     4325           
  Misses       1430     1430           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@1-leo 1-leo marked this pull request as ready for review February 12, 2026 13:59
@1-leo 1-leo requested review from frnandu and nogringo February 12, 2026 13:59
@1-leo 1-leo added the bug Something isn't working label Feb 12, 2026
@1-leo 1-leo added this to ndk-dev Feb 12, 2026
@1-leo 1-leo moved this to In Progress in ndk-dev Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant