Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Crypto] BLST pairing #4377

Merged
merged 9 commits into from
May 24, 2023
Merged

[Crypto] BLST pairing #4377

merged 9 commits into from
May 24, 2023

Conversation

tarakby
Copy link
Contributor

@tarakby tarakby commented May 23, 2023

  • introduce Fp12 to represent pairing e target group G_T elements.
  • add Fp12 comparison to the multiplicative identity element
  • implement multi_pairing function that takes two arrays of G_1 and G_2 elements and computes the product in G_T ; e(G_1[0], G_2[0]) * .. * e(G_1[n], G_2[n]) using only BLST tools (batched miller loop, final exponentiation)
  • Use the new multi_pairing for all BLS-related verifications in the package, and remove Relic pairing:
    • simple verification of one message and one key.
    • verifications of many couples (key, messages) batched per key
    • verifications of many couples (key, messages) batched per message
    • SPoCK verification
  • clean up many unused Relic functions after the changes.

@codecov-commenter
Copy link

codecov-commenter commented May 23, 2023

Codecov Report

Merging #4377 (b6b90d6) into feature/blst-based-crypto (eae8458) will increase coverage by 32.29%.
The diff coverage is n/a.

@@                      Coverage Diff                       @@
##           feature/blst-based-crypto    #4377       +/-   ##
==============================================================
+ Coverage                      18.99%   51.29%   +32.29%     
==============================================================
  Files                             52      688      +636     
  Lines                           4858    61583    +56725     
==============================================================
+ Hits                             923    31587    +30664     
- Misses                          3857    27476    +23619     
- Partials                          78     2520     +2442     
Flag Coverage Δ
unittests 51.29% <ø> (+32.29%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 636 files with indirect coverage changes

@tarakby tarakby marked this pull request as ready for review May 24, 2023 16:57
@tarakby tarakby merged commit c94b091 into feature/blst-based-crypto May 24, 2023
@tarakby tarakby deleted the tarak/blst-pairing branch May 24, 2023 17:51
This pull request was closed.
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.

2 participants