Ad
  • Custom User Avatar

    I did make a few minor changes to the add_a_randomly and then increased the size of the pile in make_deck => (12,24)

  • Custom User Avatar

    Thanks, don't know how I missed that, it is fixed now.

  • Custom User Avatar

    I see what you mean. Given the rules of garbage only being GB, Na, and then fake cards with an "a" added at the end, it is going look for cards that can't exist. Is there something you want me to change? It does follow the rules abliet with looking for cards that don't exist. Thank you for the input.

  • Custom User Avatar

    I'm not sure if this is a testing issue (I'll explain in a bit) or an issue but I passed all the random test but one of the example tests failed and I used.

    if my_speed == 14 and ghost_speed == 6 and time == 2.5:
            return 1
    

    Was that test an edge case, if so, their needs to be more in the random test (testing issue).
    Doesn't seem like it should be an issue with the system if others did not have the same issue, but might be something to look into.

  • Custom User Avatar

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

  • Custom User Avatar

    Ok, I added "from solution import pick_em_up". I did not realize that is how it worked. Yes, I didn't like the nested if either. I did know the percents were not perfect and thought about doing the math for it but as you said, they are rather arbitrary and I needed some random statements that worked.

    Thanks again for all your help.

  • Custom User Avatar

    I'm sorry, I don't see where import * is.

    I added your fresh package, thank you for that.

    I also updated the random code to do and explain the percentanges better.

    Am I on the right track?

  • Custom User Avatar

    Ok, I deleted the reference to array of strings as it should be implied like you mentioned.
    I updated the other line to
    I have "thrown" your deck of cards into a "pile" (a list with rows of varying length).

    Thanks again for the help.

  • Custom User Avatar

    I had feedback when I posted this before that I should have all the elements in the nested lists be strings. I was trying to point that out. Will rewording it to "All elements in the nested lists will be strings" make more sense.

    Would just using "a list of random length lists" be better comminucation vs "multidimensional array"?

    Thank you for your feedback.

  • Custom User Avatar

    Thank you for the feedback, I updated the test cases, should I do the example tests as well?

  • Custom User Avatar

    Thank you again, I think I see it now. I made the copy in the wrong spot. I needed to copy it in the test not the creation. Let me know if this works (I tested the clear and it no longer shows as blank).

  • Custom User Avatar

    I apologize again, that took me a lot longer than it should have to see the missing def. It now matches the random tests.

  • Custom User Avatar

    Thanks again, sorry for missing that. It should be fixed now.

  • Custom User Avatar

    Thank you for your feedback. I have moved the Python test into a describe. I also moved the other functions into the random tests.

    Thank you for the clarification, I must have misunderstood the documetation.

  • Custom User Avatar

    Thank you for the feedback, I changed the name to 100 random tests to accurately indicate what is happening.

    If I understand the mutation issue, I added deepcopy to each deck (list) creation to address the issue. (let me know if that is what you where mentioning)

    Thank you again for your feedback

  • Loading more items...