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

Homo AES compatible #343

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

xueyumusic
Copy link
Contributor

After lots of updates of HElib, it seems that homo AES has several places which are not complatible. This PR tried to make homo AES work with the latest helib.

Some places to modify are:

  1. replace L(number of level) with N(nuber of bits) in the modulus chain to be compatible with buildModChain
  2. helib namespace and include files
  3. findBaseLevel modify
  4. CMakeLists.txt modify

The running cases are:

./TEST_AES boot=1 sz=3 N=800
./TEST_AES boot=0 sz=4 N=1600

@WeiBenqiang WeiBenqiang mentioned this pull request Jul 7, 2020
@WeiBenqiang
Copy link

From my test ./TEST_AES boot=1 sz=3 N=800,for bootstapping case, N=800 is a litter big,which actural satisfies "level" we need and does not trigger batchRecrypt. And the security level is just 30. So I think we should change N to be smaller,i,e, N=500,
./TEST_AES boot=0 sz=4 N=1600,for not bootstrapping case ,N=1600 is also too large,which cannot achieve required secutiry level,just 42.7, therefore, N should be smaller, i,e, N=800 can copy with "level" we need, and security level will be 170 .
Any problem???

@faberga faberga marked this pull request as draft December 22, 2022 23:02
@faberga faberga self-assigned this Jul 15, 2023
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.

3 participants