Ad
  • Custom User Avatar

    Fixed in latest fork, all above solutions are invalid now.

  • Custom User Avatar

    Fixed in latest fork, your solution now fails.

  • Custom User Avatar

    Approved

  • Default User Avatar
  • Default User Avatar

    Thanks for the clarifcation Chrono79.
    I get the issue, made some updates and submitted successfully.

  • Custom User Avatar

    Check this out. Also, test your code with is_pangram("a"). What does it return?

    my incorrect solution passes all tests in the test mode

    There are only two tests in the Sample tests, and your code happens to pass the second one by chance.

  • Default User Avatar

    Thanks for your input. Wondering how is it possible that my incorrect solution passes all tests in the test mode (both positive & negative) but not at the time of submission. Solution also works fine on Pycharm......

  • Default User Avatar

    All tests pass in test mode but at the time of submission it fails the negative cases.

    Not sure why - is anyone facing the same or have a fix?

  • Default User Avatar

    Actually it does matter about symbols, given the code above. len(set(s.lower())) will return greater than 26 if there are non-letter symbols that cause the set to be greater than 26 in length.

    In that code, there is nothing restricting the set to contain only letters. The user posting is pointing out that the test cases should be expanded to include non-pangrams that contain many varied symbols. So the symbols do matter for appropriate test cases to pass the kata.

  • Default User Avatar

    Oh it's really good decision, but symbols don't ever matter

  • Default User Avatar

    Your code is really hard to read without proper code formatting

  • Default User Avatar

    Personally, onet thing I do when I'm not provided enough context with code wars is to try it on my own machine and try to debug it from there. It really helps.

  • Custom User Avatar

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

  • Custom User Avatar

    Mednoob, thank you for your reply!

  • Custom User Avatar

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

  • Loading more items...