Ad
  • Custom User Avatar

    Random tests are not strong enough. An inferior algorithm, hardcoding 2 fixed tests, made me pass all random tests.

  • Custom User Avatar
  • Custom User Avatar

    Before rage quitting in frustration, the user left one final mark on their account, by vandalising THIS kata, cementing their dubious legacy.

  • Custom User Avatar

    I believe your authoring priviliges should be revoked. You clearly have ZERO respect for the community, for users that have solved earlier versions of your kata's.

  • Custom User Avatar

    AND FOR HEAVEN SAKE, STOP DELETING CONTENT OF EARLIER KATA'S, EVEN IF RETIRED !!!!! This is vandalisation of content.

  • Custom User Avatar

    Please stop reposting the same kata repeatedly, making only minor tweaks each time.

  • Custom User Avatar
  • Custom User Avatar

    To be fair, I did write the Road class, so of course I know how it works. I imagine that figuring out what it does and how you can use it raises this kata to a 6kyu. I think that it is a fair rating :)

  • Custom User Avatar
    • Missing useful assertion messages (At least showing the actual input for grid sizes <= 10x10 would help solvers a lot in debugging)

    • config.truncate.threshold = 0 should be added for debugging

    • I think this part can be made more clearer, because h,w,n variables do not have much meaning in interpretation

      it(${batchSize} random tests: min/max h,w,n = [[${minRows}, ${maxRows}], [${minCols}, ${maxCols}], [${minShifts}, ${maxShifts}]]

    • [100, [1, 52], [1, 52], [1000, 101001]], should be [100, [1, 51], [1, 51], [100000, 101001]],

  • Custom User Avatar
  • Custom User Avatar

    ok, sounds fair

  • Custom User Avatar

    I misread what you meant by "2 spaces". I thought you meant before the brackets :P

    I have 0 experience in Python (my high school teaches Python, but since I'm competing for their computer science team I study strictly Java of which is the language used in the competition) so I'm just going to trust your translation and probably have my classmates and teacher run tests on it. I'll let you know if we find anything.

    I looked at the diff and I think it looks good.

    Thank you for the translation!

  • Custom User Avatar

    well, anyway, I submitted my translation

    I don't think it is needed to get the kata out of beta (which is the main reason to translate to js/py at this point) - so it may make better sense to leave it for later.
    additionally, it could be considered overengineered. that would be a valid reason to refuse it.
    or if you meant to write it yourself
    but uh I wrote it so I submitted it /shrug
    I don't like having code in preloaded. and it doesn't really need to be there, it's kind of just convenient for the sample tests to share.
    it does touch the description, so the diff would be something to look at.

    oh and.. now it's TWO spaces?

  • Custom User Avatar

    I appreciate your feedback.

    I include and mention private L and private R to let the solver know that they exist, but that to actually do anything with them they would have to use the public methods provided. It also helps them to understand that the next roads are indeed Road objects and that a Road could be thought of as a sort of Node. I feel that if I had not mentioned them, the kata would be needlessly more confusing, especially to 7kyu programmers (the target demographic of this kata). If I am to make a kata about using limited resources, I must of course give the solver adequate information about those resources.

    Though, I am not sure what you mean when you say that they are "not conceptually the same type."

    About the getters, ... I simply forgot how inheritance worked. I thought I would need them to access information about a Road when calling from other Roads, but that does not seem to be the case. Thank you for enlightening me.

    About that space issue, I will of course fix it.

  • Custom User Avatar
  • Loading more items...