#18944: Posets: maximal_chains() partial-option broken
-------------------------------------+-------------------------------------
       Reporter:  jmantysalo         |        Owner:
           Type:  defect             |       Status:  new
       Priority:  major              |    Milestone:  sage-6.8
      Component:  combinatorics      |   Resolution:
       Keywords:  posets             |    Merged in:
        Authors:  Jori Mäntysalo     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jmantysalo/maximal_chains        |  eeefd9d2c4184a3f60f505a7851ec3ff16eb7bf0
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by kdilks):

 As best I can tell, this isn't a broken function, it's bad input.
 {{{[1,2]}}} represents the two incomparable elements of
 {{{BooleanLattice(2)}}}, so they'll never be the start of a maximal chain.
 I think the better course of action would be to implement a check to
 ensure that {{{partial}}} is a saturated chain in the poset, starting with
 a minimal element.

 I think the description also needs to be clarified, because it's not clear
 just from reading it whether {{{partial=[0,1]}}} means listing all maximal
 chains with minimal element 0 or minimal element 1 (which is not what this
 does), or if it it means listing all maximal chains that have 0 as the
 minimal element and 1 as it's second element (which is what it does).

--
Ticket URL: <http://trac.sagemath.org/ticket/18944#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 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.
For more options, visit https://groups.google.com/d/optout.

Reply via email to