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

Bugfix in Poset/Lattice/Hassediagram #11189

Closed
stumpc5 opened this issue Apr 13, 2011 · 3 comments
Closed

Bugfix in Poset/Lattice/Hassediagram #11189

stumpc5 opened this issue Apr 13, 2011 · 3 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics r: duplicate t: bug

Comments

@stumpc5
Copy link
Contributor

stumpc5 commented Apr 13, 2011

This patch fixes two minor bugs in the poset code.

  1. LatticePoset should take the same input as Poset. E.g.,
sage: Poset(cover_relations=True)
Finite poset containing 0 elements
sage: LatticePoset(cover_relations=True)
...
TypeError: LatticePoset() got an unexpected keyword argument 'cover_relations'
  1. The plot for posets does not properly pass the element_label dict to HasseDiagram.plot.

Issue solved in #10998

Component: combinatorics

Keywords: poset, digraph

Author: Christian Stump

Issue created by migration from https://trac.sagemath.org/ticket/11189
@stumpc5

This comment has been minimized.

Sign in to view
@stumpc5
Copy link
Contributor Author

stumpc5 commented Apr 13, 2011

comment:1

Attachment: trac_11189-bugfix_in_poset-cs.patch.gz

@stumpc5

This comment has been minimized.

Sign in to view
@stumpc5 stumpc5 closed this as completed Apr 20, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics r: duplicate t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stumpc5