#17051: PentagonPoset and DiamondPoset, default argument for facade
-------------------------------------+-------------------------------------
Reporter: jmantysalo | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.4
Component: combinatorics | Resolution:
Keywords: | Merged in:
Authors: Jori Mäntysalo | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jmantysalo/strange_bug_in_poset_show__|
62d990cc5751d8fb6a2b71e0ac209929831b220e
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Description changed by jmantysalo:
Old description:
> This seems to trigger some kind of error:
>
> {{{
> P1=Posets.ChainPoset(3)
> P2=Posets.PentagonPoset().relabel(lambda x: x+10)
> P=Poset( (P1.list()+P2.list(), P1.cover_relations()+P2.cover_relations())
> )
> P.show()
> }}}
>
> Error message is `ValueError: element (=2) not in poset`. If I change 2.
> line to
>
> {{{
> P2=Posets.ChainPoset(5).relabel(lambda x: x+10)
> }}}
>
> error disappears.
New description:
This triggers error:
{{{
P1=Posets.ChainPoset(3)
P2=Posets.PentagonPoset().relabel(lambda x: x+10)
P=Poset( (P1.list()+P2.list(), P1.cover_relations()+P2.cover_relations())
)
P.show()
}}}
Error message is `ValueError: element (=2) not in poset`. Reason was
default value for argument `facade`. This is corrected in this ticket.
Same bug was in `Posets.DiamondPoset()`. Also Posets.DiamondPoset(2)
returns empty poset, DiamondPoset(1) gives error message. This is now
changed in for `n <= 2` they raise exception.
--
--
Ticket URL: <http://trac.sagemath.org/ticket/17051#comment:12>
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.
