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

Added ctx string to DPF computation #1146

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rozbb
Copy link
Contributor

@rozbb rozbb commented Nov 26, 2024

This adds a ctx parameter to the IDPF's gen, gen_with_random, eval, and eval_from_node methods. Addresses a subtask of #1122. Unfortunately I had to #[ignore] even more known-answer tests with this one.

Also: I tried to do more with this PR but failed each time. I wanted to do the other update which uses XofTuboShake128 for leaf nodes, but could not find a good way of abstracting away the key sizes, the reuse of the XOF for all levels but the leaf, the use of the XOF as an RngCore, etc. I think I'll leave that headache for Chris :)

src/vdaf/xof.rs Outdated Show resolved Hide resolved
src/vdaf/xof.rs Outdated Show resolved Hide resolved
src/vdaf/xof.rs Outdated Show resolved Hide resolved
src/vdaf/xof.rs Outdated Show resolved Hide resolved
@rozbb
Copy link
Contributor Author

rozbb commented Nov 26, 2024

Should be ready

src/vdaf/xof.rs Outdated Show resolved Hide resolved
src/vdaf/xof.rs Outdated Show resolved Hide resolved
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.

2 participants