6 kyu
Cure Cancer
289 of 380Jomopipi
Loading description...
Matrix
Arrays
Algorithms
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.
Hey, I have an issue and is I am not able to get the final test to work since I always get when I return the empty array, but when I add something to it there is no problem at all. I am really curious to know if I am getting something wrong and what could it be, can someone give me a little help?
see here
Thank you, I was struggling with it and you cleared my mind with that insight!
remember to close your question when it is answered ;-)
C# tests generate warnings:
The description says
But the tests expect the results to be returned in form [column, row], not [row, column].
flipped
I think that this kata should be 5 KYU
Kyus cannot be changed once the kata is approved.
nah this is more like a 7
Java, order of traversal makes a huge difference, meaning same code checking columns instead of rows timeouts and there's no indication of it anywhere.
I think that may be the intended lesson of the kata (locality of reference for row-major matrices which makes more efficient use of the cache)
This comment has been hidden.
Ok, fixed.Can you make the downvote go away? :D
Alright. All three methods pass in less than 3s now, much more reasonable. Thanks for considering my (admittedly ranty) feedback.
;D
Is it intended that the first necessary test in C# is ~300000 characters long, with many characters being null?
The tests need to be relaxed for C#. After a few timeouts, I forfeited and checked the solutions that passed. Randomly selected two user solutions that passed and they both would either time out or just barely pass at ~11.5 seconds.
Even testing the reference solution from the translator @KataSideKick yielded three consecutive timeouts.
.
C# Translation added.Please review and approve~
Note:Random test 200 -> 100
approved :)
This comment has been hidden.
It is a code optimization. You should hurry...
Thanks. A slight rearragement of conditions did the job.
that should never have passed the tests... :p
taken care of... :p
you should put a fixed test about that too, using the first line for mutation
done