Skip to content
This repository has been archived by the owner on Dec 18, 2023. It is now read-only.

Remove ATE_LOOP_COUNT_IS_NEGATIVE (Pending for the next release) #61

Closed
wants to merge 4 commits into from

Conversation

weikengchen
Copy link
Member

@weikengchen weikengchen commented May 26, 2021

Description

This PR depends on arkworks-rs/algebra#285 and therefore needs to be part of the next release. To prevent accidental merging, this is currently a draft PR.

It removes ATE_LOOP_COUNT_IS_NEGATIVE for the bn254 curve, which is also the only curve that uses BN curve parameter in arkworks.


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.
  • 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

N/A:

  • Wrote unit tests

@weikengchen weikengchen marked this pull request as draft May 26, 2021 05:37
Copy link
Contributor

@daira daira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Untested ACK

Copy link
Member

@ValarDragon ValarDragon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@weikengchen weikengchen marked this pull request as ready for review June 5, 2021 06:09
@weikengchen weikengchen mentioned this pull request Jun 6, 2021
6 tasks
@weikengchen
Copy link
Member Author

Subsumed by #62 .

@weikengchen weikengchen closed this Jun 6, 2021
@weikengchen weikengchen deleted the updated-bn-curve-trait branch June 6, 2021 20:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants