nucypher / ferveo

An implementation of a DKG protocol forked from Anoma
https://nucypher.github.io/ferveo/benchmarks/perf/tpke/index.html
GNU General Public License v3.0
5 stars 10 forks source link

Assorted chores #166

Closed piotr-roslaniec closed 10 months ago

piotr-roslaniec commented 1 year ago

Type of PR:

Required reviews:

What does it do:

Issues fixed/closed:

codecov-commenter commented 1 year ago

Codecov Report

Merging #166 (58002f5) into main (13d9d26) will increase coverage by 0.05%. The diff coverage is 97.77%.

@@            Coverage Diff             @@
##             main     #166      +/-   ##
==========================================
+ Coverage   77.95%   78.01%   +0.05%     
==========================================
  Files          23       23              
  Lines        5053     5066      +13     
==========================================
+ Hits         3939     3952      +13     
  Misses       1114     1114              
Files Coverage Δ
ferveo-common/src/keypair.rs 72.97% <100.00%> (+0.53%) :arrow_up:
ferveo-tdec/src/ciphertext.rs 97.95% <ø> (ø)
ferveo-tdec/src/combine.rs 98.51% <ø> (ø)
ferveo-tdec/src/context.rs 98.03% <ø> (ø)
ferveo-tdec/src/decryption.rs 70.72% <ø> (ø)
ferveo-tdec/src/hash_to_curve.rs 100.00% <ø> (ø)
ferveo-tdec/src/key_share.rs 32.60% <ø> (ø)
ferveo-tdec/src/lib.rs 99.78% <ø> (ø)
ferveo-tdec/src/secret_box.rs 63.15% <ø> (ø)
ferveo/src/api.rs 88.93% <100.00%> (+0.02%) :arrow_up:
... and 6 more

... and 1 file with indirect coverage changes

cygnusv commented 1 year ago

@piotr-roslaniec what do you think of ferveo-tdec instead of ferveo-tpke? I know that I suggested it, but in retrospective I don't know why I didn't think of that before!

piotr-roslaniec commented 1 year ago

@cygnusv Makes sense, also tpke doesn't quite roll off the tongue. I will update this PR shortly.