Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ChainPoset in broken for small input #8422

Closed
hivert opened this issue Mar 2, 2010 · 6 comments
Closed

ChainPoset in broken for small input #8422

hivert opened this issue Mar 2, 2010 · 6 comments
Assignees
@hivert
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-4.3.4

Comments

@hivert
Copy link

hivert commented Mar 2, 2010

For n>2 the answer is correct:

sage: Posets.ChainPoset(3).size()
3
sage: Posets.ChainPoset(4).size()
4
sage: Posets.ChainPoset(5).size()
5

However:

sage: Posets.ChainPoset(2).size()
1
sage: Posets.ChainPoset(1).size()
...
ValueError: not valid poset data.

AntichainPosets show the same wrong behavior.

Component: combinatorics

Keywords: ChainPoset

Author: Florent Hivert

Reviewer: Nicolas Borie

Merged: sage-4.3.4.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/8422
@hivert hivert added this to the sage-4.3.4 milestone Mar 2, 2010
@hivert
Copy link
Author

hivert commented Mar 2, 2010

comment:1

Attachment: trac_8422-chain_poset_fix-fh.patch.gz

@hivert
Copy link
Author

hivert commented Mar 2, 2010

Author: Florent Hivert

@hivert

This comment has been minimized.

Sign in to view
@sagetrac-nborie
Copy link
Mannequin

sagetrac-nborie mannequin commented Mar 4, 2010

Reviewer: Nicolas Borie

@sagetrac-nborie
Copy link
Mannequin

sagetrac-nborie mannequin commented Mar 4, 2010

comment:3

patch apply, doctests passed, documentation ok.

Positive review from me.

@mwhansen
Copy link
Contributor

mwhansen commented Mar 6, 2010

Merged: sage-4.3.4.alpha1

@mwhansen mwhansen closed this as completed Mar 6, 2010
@sagetrac-abmasse sagetrac-abmasse mannequin mentioned this issue Mar 8, 2010
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hivert hivert

Labels
c: combinatorics p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.3.4
Development

No branches or pull requests
2 participants
@mwhansen @hivert