#20921: LatticePoset: add function to get lattice of sublattices
-------------------------------------+-------------------------------------
       Reporter:  jmantysalo         |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-7.3
      Component:  combinatorics      |   Resolution:
       Keywords:  lattice poset      |    Merged in:
        Authors:  Jori Mäntysalo     |    Reviewers:  Travis Scrimshaw
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jmantysalo/latticeposet__add_function_to_get_lattice_of_sublattices|  
29f13a12ea8a1429196951835716a4d9181785ca
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by tscrim):

 * reviewer:   => Travis Scrimshaw


Comment:

 I would write the input section as:
 {{{
         - ``element_constructor`` -- string; can be one of the following:

           * ``'lattice'`` - (the default) elements of the lattice will
             be lattices that correspond to sublattices of ``self``
           * ``'tuple'`` - elements are tuples of elements of the
             sublattices of ``self``
           * ``'integer'`` - the elements are plain integers
 }}}
 Also, it looks like you have an unneeded import of `DiGraph`. Otherwise
 LGTM.

--
Ticket URL: <https://trac.sagemath.org/ticket/20921#comment:11>
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.

Reply via email to