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.

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

Jump to bottom

Remove Sage and add encode_agg_param #88

Merged
merged 2 commits into from
Oct 18, 2024
Merged

Remove Sage and add encode_agg_param #88

merged 2 commits into from
Oct 18, 2024

Conversation

jimouris
Copy link
Owner

partially address #87

@jimouris jimouris force-pushed the jimouris/remove-sage branch 6 times, most recently from 249b957 to a487924 Compare October 18, 2024 19:43
cjpatton
cjpatton approved these changes Oct 18, 2024
View reviewed changes
Copy link
Collaborator

@cjpatton cjpatton left a comment

Choose a reason for hiding this comment

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

👍

poc/mastic.py Outdated Show resolved Hide resolved
poc/mastic.py Outdated Show resolved Hide resolved
poc/mastic.py Outdated Show resolved Hide resolved
poc/mastic.py Outdated Show resolved Hide resolved
@jimouris jimouris merged commit c00df08 into main Oct 18, 2024
4 checks passed
@jimouris jimouris deleted the jimouris/remove-sage branch October 18, 2024 20:30
TrustworthyComputing added a commit that referenced this pull request Oct 19, 2024
@TrustworthyComputing
Squashed commit of the following:
64107bc 
commit c00df08
Author: Dimitris Mouris <[email protected]>
Date:   Fri Oct 18 16:30:23 2024 -0400

    Remove Sage and add encode_agg_param (#88)

    * Remove Sage and add encode_agg_param

    * Chris' suggestions

commit f2d9fee
Author: Christopher Patton <[email protected]>
Date:   Mon Oct 7 17:59:15 2024 -0700

    Align with draft-irtf-cfrg-vdaf-12 (*)

    * Plumb the application context string into the domain separation tag of
      each XOF invocation.

    * Change the VIDPF indices (inputs and prefixes) to `tuple[bool, ...]`,
      i.e., a sequence of bools.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cjpatton cjpatton cjpatton approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jimouris @cjpatton