You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With Lattigo v2.1.1, GenRotationKey appears to check if a RotationLeft or RotationRight key exists before generating a new one. This means that calling GenRotationkey(RotationLeft, sk, k, rotKey) twice (with the same k) will only generate one key, which seems like the correct behavior.
However, there is no such check for the Conjugate key: it is generated on every call GenRotationkey(Conjugate, ...), regardless of whether a conjugation key already exists or not. There should probably be a check like there is with RotationLeft/RotationRight so that a conjugation key is only generated once for a particular set of RotationKeys.
The text was updated successfully, but these errors were encountered:
You are absolutely correct. In fact, the evaluation-key generation interface in v2.1.1 has several other issues and we have completely revamped it (as well as the evaluator interface for Relinearization and Rotation/Conjugate) in #100. The idea behind this revamp is that the KeyGenerator interface should be independent of key-management issues such as indexing and storing keys, managing the set of rotation keys etc.
A new version tag including these changes (along with many other improvements) will be released this week.
With Lattigo v2.1.1,
GenRotationKey
appears to check if aRotationLeft
orRotationRight
key exists before generating a new one. This means that callingGenRotationkey(RotationLeft, sk, k, rotKey)
twice (with the samek
) will only generate one key, which seems like the correct behavior.However, there is no such check for the
Conjugate
key: it is generated on every callGenRotationkey(Conjugate, ...)
, regardless of whether a conjugation key already exists or not. There should probably be a check like there is withRotationLeft
/RotationRight
so that a conjugation key is only generated once for a particular set ofRotationKeys
.The text was updated successfully, but these errors were encountered: