Skip to content

Conversation

@mmagician
Copy link
Member

Description

Since BLS12 and BW6 already had final_exponentiation and multi_miller_loop as part of the {BLS12/BW6}Config, this PR adds it the same methods for BnConfig, MNT4Config and MNT6Config, for consistency.

  • some cargo fmt

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Targeted PR against correct branch (master)
  • Linked to GitHub issue with discussion and accepted design OR have an explanation in the PR that describes this work.
  • Wrote unit tests
  • Updated relevant documentation in the code
  • Added a relevant changelog entry to the Pending section in CHANGELOG.md
  • Re-reviewed Files changed in the GitHub PR explorer

@mmagician mmagician requested review from a team as code owners January 16, 2023 08:51
@mmagician mmagician requested review from Pratyush and weikengchen and removed request for a team January 16, 2023 08:51
@mmagician mmagician merged commit 24496c9 into arkworks-rs:master Jan 16, 2023
@mmagician mmagician deleted the unify-model-configs branch January 16, 2023 09:14
andrewmilson added a commit to andrewmilson/algebra that referenced this pull request Feb 9, 2023
* upstream/master:
  Add a note on `add_assign_mixed` to CHANGELOG (arkworks-rs#600)
  Fix linting on nightly Rust (arkworks-rs#598)
  Build docs correctly on docs.rs (arkworks-rs#590)
  Set correct minimum supported Rust version (arkworks-rs#592)
  Add back pairing benchmarks. (arkworks-rs#586)
  Create test-templates/README (arkworks-rs#585)
  Create README.md for `bench-templates` (arkworks-rs#583)
  Update `master` branch to 0.4.0 (arkworks-rs#582)
  Unify model configs (arkworks-rs#580)
  Correctly evaluate over domain when `self` is zero polynomial (arkworks-rs#575)
  Improve `impl From<u128> for Fp` (arkworks-rs#573)
  should just check the top bit, not two bits (arkworks-rs#560)
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