Ad
  • Custom User Avatar
  • Custom User Avatar

    mmhm, perhaps :)

  • Default User Avatar

    Could be another kata...?

  • Custom User Avatar

    I also think you enjoy generating these gravestones ;)

  • Default User Avatar

    Yeah. To confuse the devil.

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    disallowing recursive implementations because the function you require contains the word sort

    this is easy enough to work around of course. but it's very, very user-unfriendly to enforce the restrictions this way.

    there should also have been a separate test block enforcing the restrictions with normal test failures, instead of the fugly exceptions that are being raised in the course of functionality testing.

    ETA: sort_array = sorted, which is a valid solution apart from the restrictions, raises the following exception:

    TypeError: don't know how to disassemble builtin_function_or_method objects
    

    that's apparently a test error, not a code error. that's just one ( but very bad ) way in which the restrictions enforcement absolutely s*cks.

    unrelated, but I'll mention it here because the kata is retired anyway: there is absolutely no reason to use a fscking BubbleSort as the reference solution. you could just have used the native sort - it's not forbidden from testing!

    but kata depending on forbidding native methods are almost always a bad idea anyway ( which you had been told ).

  • Custom User Avatar

    modules forbidder.

    • wouldn't be enough, especially in a case like this
    • the kata is already retired
  • Custom User Avatar

    done

  • Custom User Avatar

    Already added by somebody.

  • Custom User Avatar

    I personally had to look it up.

  • Default User Avatar

    you are training on a long-outdated version of the sample tests.

  • Custom User Avatar

    This should have been an Issue. The kata is not solvable without these specifications.

  • Custom User Avatar

    I agree with your suggestion. I've now changed the output type, updated the description, and notified the three authors whose solutions were invalidated.

    Technically, the kata is now ready to be approved, but perhaps we should wait for a few more solutions and rank votes. Average assessed rank is 4kyu, but one could argue that the handling of root multiplicities pushes this closer to 3kyu territory.

  • Custom User Avatar

    Just my opinion, but given such low solve count (and the fact it's still in Beta) I'd suggest to apply this change, and leave a comment under each solution to notify and encourage them to re-submit.

  • Loading more items...