#21109: Faster meet() for lattice
-------------------------------------+-------------------------------------
Reporter: jmantysalo | Owner:
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-7.3
Component: combinatorics | Resolution:
Keywords: latticeposet | Merged in:
Authors: Jori Mäntysalo | Reviewers:
Report Upstream: N/A | Work issues:
Branch: u/jmantysalo | Commit:
/faster-meet | b679b3ddc7f7e1552a098ee5b231d5e3eeff4249
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by chapoton):
I have some questions:
I do not understand the comment {{{# T = {x_i \wedge z : z>-x_k}}}}
What is the meaning of `>-` ?
Does lc stands for "lower covers" ? if yes, that would be worth to say in
a comment
Could you take the opportunity to make the method fully pep8 compliant ?
only the line
{{{
raise ValueError("No meet for x=%s y=%s"%(x,y))
}}}
must be changed to
{{{
raise ValueError("No meet for x=%s y=%s" % (x, y))
}}}
--
Ticket URL: <https://trac.sagemath.org/ticket/21109#comment:3>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
