#12916: Dedekind-MacNeil completion of finite posets
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:  sage-combinat
  nthiery                |       Status:  needs_info
           Type:         |    Milestone:  sage-wishlist
  enhancement            |   Resolution:
       Priority:  minor  |    Merged in:
      Component:         |    Reviewers:
  combinatorics          |  Work issues:
       Keywords:         |       Commit:
  poset, lattice         |  93cefff357a7942b4c7ea539f1d97c9b0e32acb7
        Authors:         |     Stopgaps:
  Frédéric Chapoton      |
Report Upstream:  N/A    |
         Branch:         |
  u/chapoton/12916       |
   Dependencies:         |
-------------------------+-------------------------------------------------
Changes (by ncohen):

 * status:  needs_review => needs_info


Comment:

 > Yes; I would even tend to get it into Sage as a provocation: any expert
 reading the code will think "yikes, *I* can do sooo much better". And will
 do it :-)

 I am precisely the kind of guy who thought that when reading the code, and
 I don't appreciate at all to see that you consider it a game to make us
 rewrite other people's hasty code, just because we do not stand such waste
 of computation. My time is not cheaper than anybody else's.

 About this patch:

 - The name of the methods that you define is not very informative:
 `set_of_upper_bounds/set_of_lower_bounds` feels like you will get all `y`
 greater than a given `x`. It is more 'lower_bounds_of_set`, but even then
 it feels a bit awkward. Is there a word in order theory to mean "all
 elements smaller than x" ? I can't even find a function that would return
 that for such a vertex `x`

 - Computation of the set of cuts: look at the section entitled
 `Constructing the set of cuts` in the Wikipedia page that you provided.
 They say that it is equivalent to list all antichains of the Poset, and
 for that there is a `.antichains_iterator` function. (I wonder how it
 compares with
 http://www.sagemath.org/doc/reference/graphs/sage/graphs/independent_sets.html,
 but that's another subject)

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/12916#comment:7>
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