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

vdaf: Add public preprocessing of prepare messages #147

Merged
merged 1 commit into from
Jan 24, 2022

Conversation

cjpatton
Copy link
Collaborator

Aligns the API with cfrg/draft-irtf-cfrg-vdaf#14.

@cjpatton cjpatton force-pushed the cjpatton/preprocess branch 3 times, most recently from 47a02bb to c753e9b Compare January 4, 2022 21:21
@tgeoghegan
Copy link
Contributor

Could you please rebase this? I don't think commit 4406001 should be in there.

@cjpatton
Copy link
Collaborator Author

cjpatton commented Jan 4, 2022

Rebased.

src/vdaf/hits.rs Outdated Show resolved Hide resolved
src/vdaf/hits.rs Outdated Show resolved Hide resolved
src/vdaf/hits.rs Outdated Show resolved Hide resolved
src/vdaf/prio3.rs Outdated Show resolved Hide resolved
src/vdaf/hits.rs Outdated Show resolved Hide resolved
@tgeoghegan
Copy link
Contributor

Rebased on main, resolved conflicts introduced by the Hits -> poplar1 rename, and addressed my last comment. I think this is OK to merge. @cjpatton can you review?

Aligns the API with cfrg/draft-irtf-cfrg-vdaf#14.

Co-authored-by: Tim Geoghegan <timg@letsencrypt.org>
@cjpatton
Copy link
Collaborator Author

Squashed.

@tgeoghegan tgeoghegan merged commit 60f8d0c into main Jan 24, 2022
@cjpatton cjpatton deleted the cjpatton/preprocess branch January 28, 2022 23:31
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.

None yet

2 participants