{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/sympy/sympy/pull/695.diff","html_url":"https://github.com/sympy/sympy/pull/695","patch_url":"https://github.com/sympy/sympy/pull/695.patch"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/eeea4d6e4a5a7f1ed4aee90352293cce?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"eeea4d6e4a5a7f1ed4aee90352293cce","url":"https://api.github.com/users/toolforger","login":"toolforger","id":915906},"milestone":null,"assignee":null,"updated_at":"2011-11-06T23:06:36Z","labels":[],"comments":7,"closed_at":null,"url":"https://api.github.com/repos/sympy/sympy/issues/695","body":"Wherever a function accepts integers, added a test for the case that a fractional number is passed in.\r\nFixed those functions that failed, or would succeed wrongly.\r\n\r\nReworded an implementation comment to make it clearer.","html_url":"https://github.com/sympy/sympy/issues/695","title":"Added test cases for fractional numbers","number":695,"id":2130004,"created_at":"2011-11-03T07:36:22Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/0383e4cae325f65a1bbd906be4be2276?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"0383e4cae325f65a1bbd906be4be2276","url":"https://api.github.com/users/asmeurer","login":"asmeurer","id":71486},"updated_at":"2011-11-06T23:06:36Z","url":"https://api.github.com/repos/sympy/sympy/issues/comments/2648418","body":"Go ahead and fix it here. \r\n\r\nThe problem is with the argument `n` in `Sieve.__getitem__`.  Actually, all the methods of Sieve should have this `n = int(n)` kind of check (and tests added).  Some of them check for sizes, but unfortunately, this will currently lead to bugs, because we have:\r\n\r\n```pythnon\r\nIn [40]: bool(Symbol('x') > 3)\r\nOut[40]: True\r\n```\r\n\r\nAlso, I'm not a fan of the assert statements elsewhere in the file, though that's not your issue if you don't want to fix it.","id":2648418,"created_at":"2011-11-06T23:06:36Z"}}