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

flp: Make the output of Valid a vector #355

Merged
merged 1 commit into from
Jun 20, 2024
Merged

flp: Make the output of Valid a vector #355

merged 1 commit into from
Jun 20, 2024

Conversation

cjpatton
Copy link
Collaborator

@cjpatton cjpatton commented Jun 8, 2024

Closes #262.
Partially addresses #306.

If the input is valid, then each output should be zero. To save bandwidth, we generate an extra element of query randomness and use it to reduce the result.

To exercise this feature, implement a new circuit, Sum2, that generalizes Sum to support any bound, including non-powers-of-two.

@cjpatton cjpatton marked this pull request as ready for review June 8, 2024 21:48
@cjpatton cjpatton marked this pull request as draft June 11, 2024 15:53
@cjpatton
Copy link
Collaborator Author

I'm going to update this to speak in terms of "max_measurement" rather than "measurement_bound" per @divergentdave's comment here: #349 (comment)

@cjpatton cjpatton force-pushed the cjpatton/262/1 branch 2 times, most recently from a628443 to 6ada854 Compare June 11, 2024 17:48
@cjpatton cjpatton marked this pull request as ready for review June 11, 2024 17:49
@cjpatton cjpatton force-pushed the cjpatton/262/1 branch 2 times, most recently from 046c320 to dea8cb6 Compare June 12, 2024 21:28
Copy link
Contributor

@junyechen1996 junyechen1996 left a comment

Choose a reason for hiding this comment

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

LGTM overall, left some nitpicky comments.

poc/flp_generic.py Outdated Show resolved Hide resolved
poc/flp_generic.py Outdated Show resolved Hide resolved
draft-irtf-cfrg-vdaf.md Show resolved Hide resolved
draft-irtf-cfrg-vdaf.md Show resolved Hide resolved
draft-irtf-cfrg-vdaf.md Outdated Show resolved Hide resolved
draft-irtf-cfrg-vdaf.md Outdated Show resolved Hide resolved
draft-irtf-cfrg-vdaf.md Outdated Show resolved Hide resolved
poc/flp_generic.py Outdated Show resolved Hide resolved
poc/flp_generic.py Outdated Show resolved Hide resolved
poc/flp_generic.py Outdated Show resolved Hide resolved
If the input is valid, then each output should be zero. To save
bandwidth, we generate an extra element of query randomness and use it
to reduce the result.

To exercise this feature, implement a new circuit, `Sum2`, that
generalizes `Sum` to support any bound, including non-powers-of-two.

Co-authored-by: David Cook <dcook@divviup.org>
@cjpatton
Copy link
Collaborator Author

Rebased and squashed.

@cjpatton cjpatton merged commit 39fb475 into main Jun 20, 2024
6 checks passed
@divergentdave divergentdave deleted the cjpatton/262/1 branch June 20, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flp: Let the circuit output be a vector?
3 participants