Retired
Partition (retired)
Loading description...
Fundamentals
.avif)
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
Kata premise is completely flawed: Why does
partition(101, 50)expect[50, 51]only?[101]is also a valid result, sincesqrt(2n)-(3minsize)is negative in this case.Error messages are not very useful:
Shouldn't use
test.expect()I would definitively specify that the
sumof the output should be the value.I really like it otherwise, short but pleasant.
stated it explicitly in description now
test.expect(len(partition(50, 1)) >= 7, "length should be 7 at least")According to the description, the list must be [strictly] longer that 7.
corrected in description