#14180: SatSolver.clauses
--------------------------------------+-------------------------------------
Reporter: malb | Owner: jason
Type: PLEASE CHANGE | Status: needs_info
Priority: major | Milestone: sage-5.8
Component: misc | Resolution:
Keywords: sat, cryptominisat | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Martin Albrecht | Merged in:
Dependencies: #14198 | Stopgaps:
--------------------------------------+-------------------------------------
Comment (by malb):
Replying to [comment:6 ncohen]:
> HMmmmmmmmm... I've got some questions about your patch :
> * You do not seem to accept as input a DIMACS file with `is_xor` clauses
with `clauses()`, while you WRITE such files with `render_dimacs()` ?...
`clauses()` does not accept any input file, only a file to write *to* (?)
> * I do not get what this does... It's meant to store `is_xor` clauses
but I don't get it `^^;`
CryptoMiniSat expresses xor clauses by prefixing them with an "x" and
flipping the last clause if the right hand side if false.
> * I do not get this either
> Why don't you just create a gens method ? Why would you need one by
the way, as it is an alias to `clauses` anyway ?
A `gens()` method needs doctstring, doctests etc. This way, everything is
in place automatically. I added the alias `gens()` for `clauses()` because
Sage users are used to `gens()` as the method to return the generators.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14180#comment:8>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
