Opened 12 years ago

Closed 12 years ago

#8422 closed defect (fixed)

ChainPoset in broken for small input

Reported by: hivert Owned by: hivert
Priority: major Milestone: sage-4.3.4
Component: combinatorics Keywords: ChainPoset
Cc: Merged in: sage-4.3.4.alpha1
Authors: Florent Hivert Reviewers: Nicolas Borie
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by hivert)

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.

Attachments (1)

trac_8422-chain_poset_fix-fh.patch​ (2.6 KB) - added by hivert 12 years ago.

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by hivert

comment:1 Changed 12 years ago by hivert

  • Authors set to Florent Hivert
  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 12 years ago by hivert

  • Owner changed from sage-combinat to hivert

comment:3 Changed 12 years ago by nborie

  • Reviewers set to Nicolas Borie
  • Status changed from needs_review to positive_review

patch apply, doctests passed, documentation ok.

Positive review from me.

comment:4 Changed 12 years ago by mhansen

  • Merged in set to sage-4.3.4.alpha1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.