#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 ncohen):
Hello,
> 1) Can you confirm that this is right way to get maximal chains of a
poset?
Hmmmmm... Well it *would*, if `all_paths_iterator` actually did only what
it claimed (it actually returns path sorted by distance, and you do not
want that nor should you pay for that). This being said, this is precisely
the function that you should call, and if it does not do what it should,
well, that's a problem interal to `all_paths_iterator`.
2) What you think about this kind of "deprecation", where I removed broken
functionality with warning?
I do not see the point of displaying a warning before raising an
exception. Better to only raise an exception, or to remove the optiona
flag directly.
Not that "by Sage standards" this should not be done. My usual procedure
is to write to Sage devel to ask if I can remove it, have 50 persons who
usually never show up yell a me, and do nothing in the end because the
community disagrees. Pick your path.
> And for most important part, 3) what about `chain_polytope`?
I don't understand what you are talking about.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/18944#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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
