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

review strategy BNP12 #1069

Closed
obatirou opened this issue Mar 21, 2025 · 1 comment · Fixed by #1128
Closed

review strategy BNP12 #1069

obatirou opened this issue Mar 21, 2025 · 1 comment · Fixed by #1128
Assignees

Comments

@obatirou
Copy link
Collaborator

obatirou commented Mar 21, 2025

What

For now, BNP12 points are generated with points from altbn128 extended to BNF12. It will generate point with only the first coeff being 0
The strategy should generate random valid point with potentially multiple coeff being non zero.

How

Use try and increment (suggested by @zmalatrax, https://eprint.iacr.org/2009/226.pdf), find a generator or find an other way of doing it.

@github-project-automation github-project-automation bot moved this to Backlog in Keth Mar 21, 2025
@obatirou obatirou moved this from Backlog to Todo in Keth Mar 21, 2025
@obatirou obatirou changed the title review strategy BNF12 review strategy BNP12 Mar 21, 2025
@obatirou
Copy link
Collaborator Author

try and increment leads to hypothesis health checks being triggered
too_slow, large_base_example, data_too_large

@obatirou obatirou self-assigned this Mar 27, 2025
@obatirou obatirou moved this from Todo to In progress in Keth Mar 27, 2025
@obatirou obatirou moved this from In progress to In review in Keth Mar 27, 2025
@github-project-automation github-project-automation bot moved this from In review to Done in Keth Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant