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

Review/update unit tests for FIXED_NOISE_HRA and NOISE_FLOODING_HRA #791

Closed
yspolyakov opened this issue Jun 4, 2024 · 1 comment · Fixed by #805 or #810
Closed

Review/update unit tests for FIXED_NOISE_HRA and NOISE_FLOODING_HRA #791

yspolyakov opened this issue Jun 4, 2024 · 1 comment · Fixed by #805 or #810
Assignees
@pascoec
Labels
unit tests
Milestone
Release 1.2.0

Comments

@yspolyakov
Copy link
Contributor

yspolyakov commented Jun 4, 2024 •
edited by pascoec
Loading

  1. Make sure unit PRE tests are added in BGV, BFV, and CKKS for FIXED_NOISE_HRA.
  2. Check unit tests for NOISE_FLOODING_HRA (BGV only). Add tests for 1, 4 and 5 hops. Use pre-hra-secure as an example.
@yspolyakov yspolyakov added this to the Release 1.2.0 milestone Jun 4, 2024
@yspolyakov yspolyakov changed the title Add unit tests for FIXED_NOISE_HRA and NOISE_FLOODING_HRA Review/update unit tests for FIXED_NOISE_HRA and NOISE_FLOODING_HRA Jun 4, 2024
@yspolyakov yspolyakov mentioned this issue Jun 5, 2024
Merged
@pascoec pascoec linked a pull request Jun 18, 2024 that will close this issue
Issue791 #805
Merged
@pascoec pascoec reopened this Jun 19, 2024
@pascoec
Copy link
Collaborator

pascoec commented Jun 19, 2024

adding fix for 4 hops

@pascoec pascoec linked a pull request Jun 19, 2024 that will close this issue
added 4-hop test to UnitTestMultihopPRE.cpp #810
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pascoec pascoec

Labels
unit tests
Projects
None yet
Milestone
Release 1.2.0
3 participants
@dsuponitskiy @yspolyakov @pascoec