LatticePoset() arguments not forwarded to Poset()

[jm58660]

Docstring of LatticePoset() says that **options if forwarded to Poset(). It is not:

sage: P = Poset( ([0, 1,3,2, 10], [[0,1],[0,3],[0,2], [1,10],[3,10],[2,10]]), linear_extension=True)
sage: L = LatticePoset( ([0, 1,3,2, 10], [[0,1],[0,3],[0,2], [1,10],[3,10],[2,10]]), linear_extension=True)
sage: P.list()
[0, 1, 3, 2, 10]
sage: L.list()
[0, 1, 2, 3, 10]

CC: @kevindilks

Component: combinatorics

Keywords: poset

Author: Jori Mäntysalo

Branch/Commit: public/posets/forwarding_args_from_lattices-20434 @ 0fd62bc

Issue created by migration from https://trac.sagemath.org/ticket/20434

[jm58660]

comment:1

This can be seen also with plain posets:

P1 = Poset({1:[2]}, linear_extension=False, facade=False)
P2 = Poset(P1)
Q1 = Poset({1:[2]}, linear_extension=True, facade=True)
Q2 = Poset(Q1)
print P1._with_linear_extension, P1._is_facade, "-->", P2._with_linear_extension, P2._is_facade
print Q1._with_linear_extension, Q1._is_facade, "-->", Q2._with_linear_extension, Q2._is_facade

outputs

False False --> False False
True True --> False True

i.e. linear_extension=True is not copied along the data and facade argument.

I suppose that this relates to #18776.

[jm58660]

comment:2

Kevin, want to look at this?

As of my previous comment: Travis noticed at sage-devel that linear_extension=False is the default, whereas default for facade is None.

[jm58660]

Author: Jori Mäntysalo

[jm58660]

comment:3

#21666 will conflict, added as dependency.

[jm58660]

Dependencies: #21666

[jm58660]

Branch: u/jmantysalo/faster-DivisorLattice

[jm58660]

Commit: 5ae54b3

[jm58660]

comment:5

Duh. I did something really wrong with git and trac.

Waiting for 7.5 beta.

---

New commits:

8ee46bc	Modify DivisorLattice(n).
87bd385	Parentheses from import.
5ae54b3	Make LatticePoset(..., linear_extension=True) work.

[tscrim]

Changed branch from u/jmantysalo/faster-DivisorLattice to public/posets/forwarding_args_from_lattices-20434

[tscrim]

comment:6

Fixing the branch issues with #21666.

---

New commits:

0fd62bc

Make LatticePoset(..., linear_extension=True) work.

[tscrim]

Changed dependencies from #21666 to none

[tscrim]

Changed commit from 5ae54b3 to 0fd62bc

[fchapoton]

Changed keywords from none to poset

[mkoeppe]

comment:8

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.