#12848: Bug in order_ideal_complement_generators: 'down'
--------------------------------------------------------+-------------------
Reporter: nthiery | Owner:
sage-combinat
Type: defect | Status:
needs_review
Priority: major | Milestone:
sage-5.10
Component: combinatorics | Resolution:
Keywords: posets | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Nicolas M. Thiéry, Frédéric Chapoton | Merged in:
Dependencies: | Stopgaps:
--------------------------------------------------------+-------------------
Comment (by aschilling):
Hi Frédéric,
Thank you for getting this patch ready for submission!
One small technical issue is that the header of the patch needs to start
with
#14828: ... description of the patch ...
Could you change this?
I am not so happy about the naming conventions that you used. Could you
please give me references that use order filter and order ideal? Why don't
you like upper set and lower set? That seems like a standard name, see
http://en.wikipedia.org/wiki/Upper_set
Best,
Anne
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12848#comment:14>
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.
