zama-ai / tfhe-rs

TFHE-rs: A Pure Rust implementation of the TFHE Scheme for Boolean and Integer Arithmetics Over Encrypted Data.
Other
906 stars 143 forks source link

Mz/glwe ms compression #1202

Closed mayeul-zama closed 3 months ago

IceTDrinker commented 3 months ago

formatting issue is seems

For the naming the in the higher level it seems the naming could be PackedCiphertextList wdyt ?

Edit: or rather CompressedFheUintList seems CompressedFheUint is already in use

IceTDrinker commented 3 months ago

CompressedList for all APIs, you can do a commit of rename at the end to avoid effing up everything

mayeul-zama commented 3 months ago

For the naming the in the higher level it seems the naming could be PackedCiphertextList wdyt ? Edit: or rather CompressedFheUintList seems CompressedFheUint is already in use

In the integer and HL, we can mix Ints, UInts and Bools, so PackedCiphertextList seems better to me

IceTDrinker commented 3 months ago

Compressed is likely better to convey the size reduction idea if it's fine with you

IceTDrinker commented 3 months ago

do you plan on squashing the commits ? I see some mistakes, like add "hetegeneous" instead of "heterogeneous"