Skip to content

add comma for clarification of note #91

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add comma for clarification of note #91

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion tests/pyminisketch.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,7 @@ def rec_split(poly, randv):
# Try consecutive randomization factors randv, until one is found that factors poly.
while True:
# Compute the trace of (randv*x) mod poly. This is a polynomial that maps half of the
# domain to 0, and the other half to 1. Which half that is is controlled by randv.
# domain to 0, and the other half to 1. Which half that is, is controlled by randv.
# By taking it modulo poly, we only add a multiple of poly. Thus the result has at least
# the shared roots of the trace polynomial and poly still, but may have others.
trace = poly_tracemod(poly, randv, gf)
Expand Down