Ad
  • Default User Avatar

    nvm I forked it myself.

  • Default User Avatar

    Ok.. uhm.
    so what you did might be technically correct, but .. could you please write it how it is in python? because what you wrote looks like you gave up on implementing it as intended.

    also, you seem to have edited the already approved translation, normally the button to edit that is not even visible so I'm not sure how you even did that, maybe you manually added /edit to the url. regardless of how that happened, that change isn't live.

  • Default User Avatar

    I don't see any further details I could share. Those are the details. Your test gen generates cases that do not have a solution.

    Which is to say, they do not satisfy the property in the kata description

    I have four positive integers, A, B, C and D, where A < B < C < D. The input is a list of the integers A, B, C, D, AxB, BxC, CxD, DxA in some order.

    instead it is:

    picking 4 random values that don't relate to each other

    for ( int i = 0; i < 4; i++ ) my_list.add((int)(Math.random()*range) + 1);
    

    there are examples of such invalid test cases in the open issue above, another example of an invalid test case that your test gen may generate is [1, 1, 1, 1, 1, 1, 1, 1]

  • Default User Avatar

    @Featres test gen generates invalid input, note requirements in description on input and lack of enforcing that in the gen code, instead picking 4 random values that don't relate to each other.

    .. last seen december, ~2months ago, though.

    kinda wish we had bit of an ownership/accountability/nosy system going on with notifications and such, instead of like here, having people not notice issues popping up within one month on their translations/approvals.

  • Custom User Avatar

    Raw HTML is discouraged, using Markdown should be preferred: [link title](link url).

  • Custom User Avatar

    <a href="blablabla">lorem lorem</a>