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: |
Description (last modified by )
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
comment:2 Changed 7 years ago by
- Owner changed from z to ferriszorro
Changed 7 years ago by
#!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
- 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
- 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
This being said this bug report is a real bug report O_o
comment:6 Changed 7 years ago by
- 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
Partitions error