New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default facade=True for order_ideals_lattice() #19265
Comments
comment:2
Logical change, I think. Nathann, a review? New commits:
|
Commit: |
comment:3
Hmmmmmm... I can't say that I don't want to see the end of these non-facade things, but I am not sure that I should be the one to positive-review them, especially for this reason. In the present case, however, I'd say that the logical change would be to define the default of i.e. if you wanted a non-facade poset, you probably want a non-facade order ideal. I add frederic in Cc, he might have an opinion on this. Nathann |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:5
Replying to @nathanncohen:
You are right. I made a new commit according to that. |
Reviewer: Frédéric Chapoton |
comment:6
lgtm |
Changed branch from u/jmantysalo/default_facade_true_for_order_ideals_lattice__ to |
facade=True
is default for posets in general, and at least #17051 was accepted. Hence I suggest changingorder_ideals_lattice()
to also have that same default.CC: @nathanncohen @fchapoton
Component: combinatorics
Author: Jori Mäntysalo
Branch/Commit:
097ac4e
Reviewer: Frédéric Chapoton
Issue created by migration from https://trac.sagemath.org/ticket/19265
The text was updated successfully, but these errors were encountered: