Opened 7 years ago

Last modified 7 years ago

#17548 closed defect

Partitions() involving min_slope argument is buggy — at Version 6

Reported by: ferriszorro Owned by: ferriszorro
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: combinatorics Keywords: Partitions
Cc: jakobkroeker, aschilling Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by tmonteil)

The following bug was reported by an humanoid:

sage: Partitions(5, min_slope=1).list()
ValueError: [2, 4] is not a valid partition

Even worse (since silent):

sage: Partitions(5, min_slope=2).list()
[[5]]

Or

Change History (7)

comment:1 Changed 7 years ago by ferriszorro

Partitions error

comment:2 Changed 7 years ago by ferriszorro

  • Owner changed from z to ferriszorro

Changed 7 years ago by ferriszorro

#!diff # HG changeset patch # Date 1300721936 0 foo diff --git a/bar b/bar --- a/bar +++ b/bar @@ -1,2 +1,3 @@ baz quux +xyzzy

comment:3 Changed 7 years ago by chapoton

  • Authors z deleted
  • Branch v5.7 deleted
  • Dependencies sage 5.7 deleted
  • Merged in sage5.7 deleted
  • Milestone changed from sage-feature to sage-duplicate/invalid/wontfix
  • Reviewers f deleted
  • Status changed from new to needs_review

comment:4 Changed 7 years ago by ncohen

  • Status changed from needs_review to positive_review

Frédéric: the wontfix tickets should be set to 'positive review', otherwise Volker will not see them.

"Merry Christmas",

Nathann

comment:5 Changed 7 years ago by ncohen

This being said this bug report is a real bug report O_o

comment:6 Changed 7 years ago by tmonteil

  • Component changed from number theory to combinatorics
  • Description modified (diff)
  • Milestone changed from sage-duplicate/invalid/wontfix to sage-6.5
  • Status changed from positive_review to needs_work
  • Summary changed from sage5.7 Partitions() error to Partitions() involving min_slope argument is buggy

I was very puzzled when i saw this ticket, since my Turing tests were not able to conclude (unless we just reached the singularity). I propose to reopen the ticket since the report is correct.

By the way, i am not sure the following is a correct behaviour:

sage: P = Partitions(5, min_slope=2)
sage: Q = Partitions(5)
sage: Q[0] 
[5]
sage: Q[0] in P
False
Note: See TracTickets for help on using tickets.