Ad
  • Default User Avatar

    Should be all fixed now. Note that int changed to long, so you'll need to change that.

  • Default User Avatar

    (cw doesn't send out notifications in this scenario so I didn't see your messages until now)

    Yeah the issue isn't solved. But it's a duplicate. So the issue is still open. And it's being worked on. Probably. The translation author did poke at it but it needs another pass and uh. The point is, the issue isn't being ignored.

    I think you're wrong about the size being different from 8 though. Maybe you're printing it in some weird way that you're looking at two of them. Java people seem to be prone to that kind of thing, while for example python/js people can just print it and get it neatly formatted and therefore won't mix them up that way. (not that it matters until the other issue gets fixed)

  • Default User Avatar

    So the issue still is not solved.

  • 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

    Tried it. First of all the size of the initial array is more than 8 but in the description it says it is 8. Second, it expects and returns a number that is not even in the initial given array.

  • Default User Avatar

    @natan, could you explain in a little more detail what the problem is?

  • Default User Avatar

    by printing. please don't open issues without having fully worked out what's going on. if there already is an issue that you think matches, don't create a duplicate.

  • Default User Avatar

    yes, but, duplicate issue

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

  • Default User Avatar

    I do not know. Is there a way to see the random tests' inputs? Because I can only see the results and the errors, NOT the inputs of the random tests. And by the way only random tests don't pass. All the other tests pass. And as I can see in this discussion many other people had the same issue like I have.

  • Default User Avatar

    what was the test then?

    not passing tests is in itself normal.

  • Default User Avatar

    Java

    All tests pass except random tests. Think there is an issue with the condition in random tests that doesn't match the condition described in the instruction.

  • Default User Avatar

    using Java by the way.

  • Default User Avatar

    guys can pass all tests except random ones. Any ideas what to do?

  • Default User Avatar

    One way is to look for a formula that, given i and j, returns the value in the array at position i,j. In row r, the elements increase from 1 to r in the first r columns, remain at r until column r from the end, and then decrease to 1. By comparing the row and column value appropriately, you can figure out what the array element has to be.

  • Loading more items...