#13558: Add Philipp Jovanovic ANF to CNF converter
-------------------------------+--------------------------------------------
       Reporter:  malb         |         Owner:  jason   
           Type:  enhancement  |        Status:  new     
       Priority:  major        |     Milestone:  sage-5.5
      Component:  misc         |    Resolution:          
       Keywords:  SAT          |   Work issues:          
Report Upstream:  N/A          |     Reviewers:          
        Authors:               |     Merged in:          
   Dependencies:               |      Stopgaps:          
-------------------------------+--------------------------------------------

Comment (by malb):

 Replying to [comment:4 cosh]:
 >  * Every function now has at least one doctest

 Note that you misformated the doctest blocks: There's a {{{::}}} after the
 preceding line (e.g. {{{EXAMPLE::}}}) which starts a code block. Also,
 there's supposed to be an empty line between the {{{EXAMPLE::}}} and the
 first line of the code block. Again, this is part of the markup which
 starts a code block.

 >  * `phi` is now a `@property`
 >  * marked `self.one` and `self.zero` as local
 (although `self.zero` could be deleted completely as it is never used)

 Lets remove it then :)

 > What I want to do next is adding some error checking mechanisms for user
 parameters and re-implementing the `c_lin_cnf` function in a more generic
 way. Anything else that I forgot?

 As far as I can see, no.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13558#comment:5>
Sage <http://www.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.

Reply via email to