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

Renamed size to cardinality in poset #8735

Closed
hivert opened this issue Apr 21, 2010 · 8 comments
Closed

Renamed size to cardinality in poset #8735

hivert opened this issue Apr 21, 2010 · 8 comments
Assignees
@hivert
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-4.4.1

Comments

@hivert
Copy link

hivert commented Apr 21, 2010

The cardinality of a set was standardized as cardinality. Posets where forgotten when standardization ocured.

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: posets cardinality

Author: Florent Hivert

Reviewer: Nicolas Thiery

Merged: sage-4.4.1.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/8735
@hivert hivert added this to the sage-4.4.1 milestone Apr 21, 2010
@hivert hivert self-assigned this Apr 21, 2010
@hivert
Copy link
Author

hivert commented Apr 21, 2010

Attachment: trac_8735-poset_size_cardinality-fh.patch.gz

@hivert
Copy link
Author

hivert commented Apr 21, 2010

comment:1

For the info: I got a all test passed on sage.

@hivert
Copy link
Author

hivert commented Apr 21, 2010

Reviewer: Nicolas Thiery

@nthiery
Copy link
Contributor

nthiery commented Apr 21, 2010

comment:2

Thanks for running the tests; since massena is out, I am a bit stuck on running tests ...

This change was needed, and the implementation looks right. Positive review!

Thanks for spotting and handling this!

@nthiery

This comment has been minimized.

Sign in to view
@williamstein
Copy link
Contributor

Merged: 4.4.1.alpha2

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 3, 2010

Changed merged from 4.4.1.alpha2 to sage-4.4.1.alpha2

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Apr 18, 2011

comment:5

See follow-up at #11214.

@rbeezer rbeezer mannequin mentioned this issue Apr 21, 2011
Closed
@a-andre a-andre mentioned this issue Dec 19, 2014
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.4.1
Development

No branches or pull requests
3 participants
@nthiery @williamstein @hivert