Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Ring-) Learning with Errors Oracles #14478

Closed
malb opened this issue Apr 23, 2013 · 12 comments
Closed

(Ring-) Learning with Errors Oracles #14478

malb opened this issue Apr 23, 2013 · 12 comments

Comments

@malb
Copy link
Member

malb commented Apr 23, 2013

The (Ring-)LWE problem is a central problem in cryptography these days, but generators for LWE instances are not easily available. At

https://bitbucket.org/malb/lwe-generator/

a family of LWE generators is implemented.

We should include this in Sage.

Component: cryptography

Author: Martin Albrecht, Robert Fitzpatrick, Daniel Cabracas, Florian Göpfert, Michael Schneider

Reviewer: Minh Van Nguyen

Merged: sage-5.10.beta3

Issue created by migration from https://trac.sagemath.org/ticket/14478

@malb
Copy link
Member Author

malb commented Apr 23, 2013

comment:2

Note to reviewer: we cross-refereed the correctness of parameter choices, so the main task would probably be to check that the patch is up to Sage's formal standards.

@sagetrac-Bouillaguet
Copy link
Mannequin

sagetrac-Bouillaguet mannequin commented Apr 24, 2013

comment:3

Hi Martin & friends, I'm on it. Give me a day or so.

@malb
Copy link
Member Author

malb commented Apr 25, 2013

comment:4

I've updated the patch

  • adding the module to the reference manual
  • fixed cosmetic issues in the HTML output
  • fixed bug where m wasn't restricted where it should

See commits

@malb
Copy link
Member Author

malb commented May 5, 2013

comment:5

Attachment: trac_14478_lwe.patch.gz

fixed doctest failure.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 7, 2013

Attachment: trac_14478_reviewer.patch.gz

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 7, 2013

comment:6

It looks OK. The reviewer patch fixes some typos.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 7, 2013

Reviewer: Minh Van Nguyen

@malb
Copy link
Member Author

malb commented May 8, 2013

comment:8

Reviewer patch looks fine, thanks Minh!

@jdemeyer
Copy link

jdemeyer commented May 8, 2013

comment:9
dochtml.log:[cryptogra] /mazur/release/merger/sage-5.10.beta3/local/lib/python2.7/site-packages/sage/crypto/lwe.py:docstring of sage.crypto.lwe:4: WARNING: Bullet list ends without a blank line; unexpected unindent.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 8, 2013

comment:10

Attachment: trac_14478_fix-indent.patch.gz

The indentation should be fixed now.

@malb
Copy link
Member Author

malb commented May 9, 2013

comment:11

I rebuilt the docs with the three patches applied and saw no warning.

@jdemeyer
Copy link

Merged: sage-5.10.beta3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants