lattice-estimator
lattice-estimator copied to clipboard
Comment difference rough vs estimate
Following a private email discussion with Martin and others, I would suggest adding a comment in the README about the main API and the cost models in use throughout the codebase.
I was quite confused that rough uses [ADPS16] by default, but the rest of the code rather relies on [MATZOV] for the concrete cost of lattice reduction.
Thanks for your feedback! I updated the pull request to take it into account, I hope that's better now :)
Great! A few CI failures to take care of but otherwise good to go. Thank you!
Perfect! Should be good now
Ta!