{"action":"closed","issue":{"url":"https://api.github.com/repos/jonmcalder/refactor/issues/1","repository_url":"https://api.github.com/repos/jonmcalder/refactor","labels_url":"https://api.github.com/repos/jonmcalder/refactor/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/jonmcalder/refactor/issues/1/comments","events_url":"https://api.github.com/repos/jonmcalder/refactor/issues/1/events","html_url":"https://github.com/jonmcalder/refactor/issues/1","id":174961322,"number":1,"title":"include.lowest with inconsistent default","user":{"login":"lorwal","id":10477073,"avatar_url":"https://avatars.githubusercontent.com/u/10477073?v=3","gravatar_id":"","url":"https://api.github.com/users/lorwal","html_url":"https://github.com/lorwal","followers_url":"https://api.github.com/users/lorwal/followers","following_url":"https://api.github.com/users/lorwal/following{/other_user}","gists_url":"https://api.github.com/users/lorwal/gists{/gist_id}","starred_url":"https://api.github.com/users/lorwal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lorwal/subscriptions","organizations_url":"https://api.github.com/users/lorwal/orgs","repos_url":"https://api.github.com/users/lorwal/repos","events_url":"https://api.github.com/users/lorwal/events{/privacy}","received_events_url":"https://api.github.com/users/lorwal/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-09-04T17:57:55Z","updated_at":"2016-09-05T08:37:27Z","closed_at":"2016-09-05T08:37:27Z","body":"## Problem\r\nDepending on how breaks are specified, lowest values are included or not. \r\n\r\n* `cut(sample(10), breaks = 2)` gives the levels 1-5 and 6-10 (because the default is overwritten internally on line 78, while\r\n* `cut(sample(10), breaks = c(1, 5, 10)`gives the levels 2-5 and 6-10.\r\n\r\n## Suggested solution\r\nProbably due to the default in cut.default, you wanted to keep include.lowest = FALSE. I think it is a counter intuitive default, so let's do away with it and change it to TRUE and hence remove line 78.\r\n"}}
{"actor":{"display_login":"lorwal"}}