#6514: [with patch, needs review] Boolean function for cryptography
--------------------------+-------------------------------------------------
Reporter: ylchapuy | Owner: somebody
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1.2
Component: cryptography | Keywords: boolean function, cryptography
Reviewer: | Author: Yann Laigle-Chapuy
Merged: |
--------------------------+-------------------------------------------------
Comment(by malb):
Hi, even I don't get some of my earlier comments. Sorry, I was in a rush.
* it would be nice to have docs for walsh, yellow_code, reed_muller
* wouldn't it be more appropriate if {{{cdef _walsh_transform}}} was a
tuple instead of a list?
* the patch applies cleanly
* doctests pass
* you could consider breaking lines at 80 or 120 characters in the
docstrings maybe
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6514#comment:7>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
