Ad
  • 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.

  • Custom User Avatar

    It was a really nice kata. Unfortunately, the author hasn't been around since December 2022. I could change the output type from float to Fraction, but it would invalidate all existing five solution (including the author's). I'm not sure what the best way to proceed is.

  • Custom User Avatar

    Only invalidated solutions were the ones which tried to use a correct prime checker and then append the wrong results as special cases.

  • Custom User Avatar

    Okay I have approved your fork with some modifications. If this invalidates solutions then it's on me.

  • Custom User Avatar

    You shouldn't end up invalidating any solutions if you add the import to the setup as well as preloaded.

    It'll be disgusting, yes, but we haven't been given much choice.

  • Default User Avatar

    My apologies for not responding to your original suggestion. Obviously the question of whether indexes should start at 0 or 1 is a matter of style. I prefer 1 in this case for the following reason: If someone says something like "the maximizing move was move 8", they most likely mean the 8th one, not the 9th one.

  • Custom User Avatar

    I don't know whether we should still make such a breaking change after approval of the kata.

  • Custom User Avatar
  • Custom User Avatar

    I've made a fork fixing this issue by using an exterenal library.

    Honestly, I believe it would be better to explicitly import it in the initial solution and not mention it in the description, as that would make it easier to translate. However, as to not invalidate any solutions, I just messed with the preloaded section.

  • Default User Avatar

    Which makes no practical sense, does it?

  • Default User Avatar

    Random test has been updated. Thanks for contributing.

  • Loading more items...