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

Adding is_lattice to poset #16873

Closed
jm58660 mannequin opened this issue Aug 24, 2014 · 7 comments
Closed

Adding is_lattice to poset #16873

jm58660 mannequin opened this issue Aug 24, 2014 · 7 comments
Assignees
@jm58660
Labels
c: combinatorics r: duplicate t: enhancement

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Aug 24, 2014

Posets class contains all three: top(), bottom() and is_bounded(). However, for now there is no is_lattice() even there exists is_join_semilattice() and is_meet_semilattice().

Component: combinatorics

Keywords: poset

Branch/Commit: u/jmantysalo/adding_is_lattice_to_poset @ a7b214f

Issue created by migration from https://trac.sagemath.org/ticket/16873
@jm58660 jm58660 mannequin added this to the sage-6.3 milestone Aug 24, 2014
@jm58660 jm58660 mannequin added the p: major / 3 label Aug 24, 2014
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2014

Changed keywords from none to poset

@jm58660 jm58660 mannequin added c: combinatorics and removed p: major / 3 labels Aug 24, 2014
@jm58660 jm58660 mannequin modified the milestones: sage-6.3, sage-6.4 Aug 24, 2014
@jm58660 jm58660 mannequin added the t: enhancement label Aug 24, 2014
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2014

Branch: u/jmantysalo/adding_is_lattice_to_poset

@jm58660

This comment has been minimized.

Sign in to view
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2014

Commit: 79d9c2f

@jm58660 jm58660 mannequin self-assigned this Aug 24, 2014
@jm58660 jm58660 mannequin added the s: needs review label Aug 24, 2014
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 24, 2014

Branch pushed to git repo; I updated commit sha1. New commits:

a7b214fAdded is_lattice() method.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 24, 2014

Changed commit from 79d9c2f to a7b214f

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2014

comment:7

Arghs. is_lattice() already exists on categories/finite_posets.py. Hence this ticket can be closed. (And another one opened for documentation of posets...)

@jm58660 jm58660 mannequin removed this from the sage-6.4 milestone Aug 24, 2014
@vbraun vbraun closed this as completed Aug 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jm58660 jm58660

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

No branches or pull requests
1 participant
@vbraun