6 kyu
Clustering corrupted banks
73 of 74Donvito-911
Loading description...
Data Structures
Algorithms
Set Theory
View
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.
Factor translation
Factor doesn't seem to handle giant hashtables as well as Python does, so I've slightly relaxed the big tests to allow the same kinds of solutions to pass.
Please review and approve.
Could use a tag:
set theory
tag added
Hi,
Suggestion: you should emphasize the
Assumptions
word in the description (more than it is now)Cheers
Hello, how do you recommend me to emphasize? for me it is clear but two issues already about it. Perhaps giving an example of an invalid input?
Just something like :
## ___<u>Assumptions</u>___
Note that one actual problem (but not your fault) is that the numbers in the example at the end of that section use a bigger font size. So they litteraly draw in the eye of the reader, leading them to skip the two bullet points (that's what happened for me...). Maybe try using a simple code span instead of KatTex notation?
Thanks, the kata has been updated. I hope it is clearer now.
(note: you'll have to handle the last issue, then close it, then publish the kata again, to make it available)
warning, the fixed tests are in a describe block labelled "random"... ;)
sorry, now I guess it is published and fixed this minor issues.
Modifying the input in user function will break the random tests.
You should either pass a copy of the list to user function, or run the reference solution before the user function.
Kata updated.
There is nothing in the kata description to restrict inputs such as
[(1, 3), (2, 3), (3, 1)]
, which contains banks taking multiple or no loans. This will break the entire premise of the kata.there is this:
Has it been added after your message?
Hi, the description has been always there. The assumptions actually restrict the inputs since it is assuming bijections. I'll mark the issue as resolved but if you dont agree please let me now.