google / google/heir

Clean up LWECiphertext Attributes

Open
#2,628 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
MLIR
Stars
906
Forks
171
Avg merge
4d 12h
Merged PRs (30d)
32

Description

In 2/9 office hours, we discussed a pain point related to KeySwitchKey types: since they have additional "non-level" moduli, the ModulusChain attribute doesn't really work well for them. An LWEPublicKey would be more accurate, but we're missing lots of ops on PublicKeys that we would have to add.

A better approach would be to remove unnecessary attributes from LWECiphertexts. We think that PlaintextSpace, ModulusChain, and ApplicationData attributes are probably no longer needed. For this ticket: validate this hunch, remove unneeded LWECiphertext attributes, and either unify LWECiphertext with LWEPublicKey or make them interoperable.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.