Ad
  • Custom User Avatar

    Solution setup missing import for Fraction

    Tests need updating to use new framework etc.

  • 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

    I fixed the description to reflect this requirement.

  • Custom User Avatar

    Very nice idea!!

    But since Python has support for it, the output should a list of rationals

  • Default User Avatar

    It should be mentioned in the description that you may need to include the same zero multiple times in the result. I assume the task was just to find rational zeros, without caring for their multiplicities.

  • Default User Avatar

    Just getting started, but I did not see a mention of what degree the polynomials would be. I assume only deg <= 2 by the test cases?

  • Default User Avatar

    Added random tests with more complicated rational roots and a more complex multiplicity test (the random tests also include examples with complex multiplicities)

  • Custom User Avatar
    • Needs random tests
    • Needs polynomials that returns more complicated rational roots (e.g 1/7, or something like 50/1001)
    • Needs more sophisticated tests on multiplicities (e.g polynomial with roots [0,0,1])