Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

uniqueN is not handling NULL value #1429

Closed
jangorecki opened this issue Nov 10, 2015 · 2 comments
Closed

uniqueN is not handling NULL value #1429

jangorecki opened this issue Nov 10, 2015 · 2 comments
Assignees
@arunsrinivasan
Labels
bug
Milestone
v1.9.8

Comments

@jangorecki
Copy link
Member

Latest 1.9.7:

library(data.table)
uniqueN(NULL)
# Error in forderv(x, by = by, retGrp = TRUE) : 
#    First column being ordered is type 'NULL', not yet supported
arunsrinivasan added a commit that referenced this issue Nov 19, 2015
@arunsrinivasan
Merge pull request #1430 from jangorecki/uniquen-null
4fdb4ec 
uniqueN handle NULL, closes #1429
@arunsrinivasan
Copy link
Member

There's a bug when uniqueN() is used along with a list() at the moment.. it results in length(unique(.)) which would return the wrong result.

@arunsrinivasan arunsrinivasan added this to the v1.9.8 milestone Feb 26, 2016
@arunsrinivasan arunsrinivasan self-assigned this Feb 26, 2016
@arunsrinivasan
Copy link
Member

This bug is as a result of #1224, not this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arunsrinivasan arunsrinivasan

Labels
bug
Projects
None yet
Milestone
v1.9.8
Development

No branches or pull requests
2 participants
@arunsrinivasan @jangorecki