{"action":"created","issue":{"url":"https://api.github.com/repos/randomchars42/bioset/issues/1","repository_url":"https://api.github.com/repos/randomchars42/bioset","labels_url":"https://api.github.com/repos/randomchars42/bioset/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/randomchars42/bioset/issues/1/comments","events_url":"https://api.github.com/repos/randomchars42/bioset/issues/1/events","html_url":"https://github.com/randomchars42/bioset/issues/1","id":313499164,"number":1,"title":"Avoid redefining n()","user":{"login":"krlmlr","id":1741643,"avatar_url":"https://avatars0.githubusercontent.com/u/1741643?v=4","gravatar_id":"","url":"https://api.github.com/users/krlmlr","html_url":"https://github.com/krlmlr","followers_url":"https://api.github.com/users/krlmlr/followers","following_url":"https://api.github.com/users/krlmlr/following{/other_user}","gists_url":"https://api.github.com/users/krlmlr/gists{/gist_id}","starred_url":"https://api.github.com/users/krlmlr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krlmlr/subscriptions","organizations_url":"https://api.github.com/users/krlmlr/orgs","repos_url":"https://api.github.com/users/krlmlr/repos","events_url":"https://api.github.com/users/krlmlr/events{/privacy}","received_events_url":"https://api.github.com/users/krlmlr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-04-11T21:44:08Z","updated_at":"2018-04-17T14:04:37Z","closed_at":null,"author_association":"NONE","body":"The release candidate of dplyr 0.7.5 will use the correct implementation of `n()` only if this function isn't overridden. Most likely because of this, CRAN checks of bioset fail now: https://github.com/tidyverse/dplyr/blob/master/revdep/new-problems.md#bioset\r\n\r\nA solution seems to be to assign `dplyr::n` to the local variant of `n`, or to avoid overriding `n`."},"comment":{"url":"https://api.github.com/repos/randomchars42/bioset/issues/comments/382004589","html_url":"https://github.com/randomchars42/bioset/issues/1#issuecomment-382004589","issue_url":"https://api.github.com/repos/randomchars42/bioset/issues/1","id":382004589,"user":{"login":"randomchars42","id":10254344,"avatar_url":"https://avatars0.githubusercontent.com/u/10254344?v=4","gravatar_id":"","url":"https://api.github.com/users/randomchars42","html_url":"https://github.com/randomchars42","followers_url":"https://api.github.com/users/randomchars42/followers","following_url":"https://api.github.com/users/randomchars42/following{/other_user}","gists_url":"https://api.github.com/users/randomchars42/gists{/gist_id}","starred_url":"https://api.github.com/users/randomchars42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/randomchars42/subscriptions","organizations_url":"https://api.github.com/users/randomchars42/orgs","repos_url":"https://api.github.com/users/randomchars42/repos","events_url":"https://api.github.com/users/randomchars42/events{/privacy}","received_events_url":"https://api.github.com/users/randomchars42/received_events","type":"User","site_admin":false},"created_at":"2018-04-17T14:04:37Z","updated_at":"2018-04-17T14:04:37Z","author_association":"OWNER","body":"Hey krlmlr,\r\n\r\nif I undestand you correctly I need to add to the following existing code\r\n\r\n```\r\n    data <- data %>%\r\n      dplyr::group_by(!! ids) %>%\r\n      dplyr::mutate(\r\n        !! target_n := n(),\r\n        !! target_mean := mean(!! target),\r\n        !! target_sd := stats::sd(!! target),\r\n        !! target_cv := stats::sd(!! target) / mean(!! target)\r\n      ) %>%\r\n      dplyr::ungroup()\r\n```\r\nsomething like:\r\n\r\n```\r\nn <- dplyr::n\r\n```\r\n\r\n?\r\n\r\nI'm not aware that I redefine / override `n()` anywhere explicitly or implicitly, would you please be so kind as to explain what might trigger this new error?\r\n"}}
{"actor":{"display_login":"randomchars42"}}