Closed b-kamphorst closed 3 years ago
Merging #18 into master will decrease coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #18 +/- ##
==========================================
- Coverage 90.42% 90.41% -0.01%
==========================================
Files 12 12
Lines 3999 3996 -3
==========================================
- Hits 3616 3613 -3
Misses 383 383
Impacted Files | Coverage Δ | |
---|---|---|
mpyc/runtime.py | 86.06% <100.00%> (-0.03%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 1671a7f...88c0150. Read the comment docs.
Yes, thanks a lot! That is a nice improvement, good thinking.
Tested the change also with the demos and found no problems.
An earlier pull request fixed the conversion from SecFld to SecFxp/SecInt for signed secrets, but in a very inefficient way. It turns out that the fix could have been much easier and without need for a secure conversion.