Crypto-TII / CryptographicEstimators

This project gathers and standardize command line scripts to estimate the difficulty of solving hard mathematical problems related to cryptography.
https://estimators.crypto.tii.ae/
GNU General Public License v3.0
34 stars 5 forks source link

refactor: remove sage from mr estimator #199

Closed Dioprz closed 2 weeks ago

Dioprz commented 2 weeks ago

Description

About performance:

This implemention of minors_polynomials[_degree] is about 8x times slower than the Sage-based one. With that said, we will be able to replace Sympy with python-flint in the future by using a C-Flint type used to calculate determinants over matrices of polynomials, when ported and exposed by python-flint. This should resolve (and probably even improve) the performance with respect to Sage.

Review process

make docker-run
pytest --doctest-modules -n auto -vv -s cryptographic_estimators/MREstimator/

Pre-approval checklist

sonarcloud[bot] commented 2 weeks ago

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
53.8% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud