Ticket #8735 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Renamed size to cardinality in poset

Reported by: hivert Owned by: hivert
Priority: major Milestone: sage-4.4.1
Component: combinatorics Keywords: posets cardinality
Cc: sage-combinat Work issues:
Report Upstream: N/A Reviewers: Nicolas Thiery
Authors: Florent Hivert Merged in: sage-4.4.1.alpha2
Dependencies: Stopgaps:

Description (last modified by nthiery) (diff)

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

Attachments

trac_8735-poset_size_cardinality-fh.patch Download (4.3 KB) - added by hivert 3 years ago.

Change History

Changed 3 years ago by hivert

comment:1 Changed 3 years ago by hivert

  • Status changed from new to needs_review
  • Reviewers set to Nicolas Thiery

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

comment:2 Changed 3 years ago by nthiery

  • Status changed from needs_review to positive_review
  • Description modified (diff)

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!

comment:3 Changed 3 years ago by was

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to 4.4.1.alpha2

comment:4 Changed 3 years ago by mvngu

  • Merged in changed from 4.4.1.alpha2 to sage-4.4.1.alpha2

comment:5 Changed 2 years ago by rbeezer

See follow-up at #11214.

Note: See TracTickets for help on using tickets.