8000 [R-Forge #2422] Handle columns with complex numbers · Issue #602 · Rdatatable/data.table · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[R-Forge #2422] Handle columns with complex numbers #602

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
Closed
arunsrinivasan opened this issue Jun 8, 2014 · 3 comments
Closed

[R-Forge #2422] Handle columns with complex numbers #602

arunsrinivasan opened this issue Jun 8, 2014 · 3 comments
Labels
feature request

Comments

@arunsrinivasan
Copy link
Member

Submitted by: Paul Metzner; Assigned to: Nobody; R-Forge link

Necessary to use data.table for signal processing, especially for the analysis of power spectra.
@richierocks richierocks mentioned this issue Dec 10, 2014
Closed
@MichaelChirico
Copy link
Member
MichaelChirico commented Mar 18, 2016 •
edited
Loading

I suppose this issue is open because complex columns are not feature-complete?

E.g,

set.seed(102439)
DT <- data.table(C = c(1+2i, 1+2i, 1-2i, 1-2i), 
                 N = runif(4), I = c(1L, 1L, 2L, 2L))
DT[ , .N, by = C]

Error in``[.data.table(DT, , .N, by = C):
column or expression 1 of `by` or `keyby` is type `complex`. Do not quote column names. Usage: `DT[,sum(colC),by=list(colA,month(colB))]`

Other standard operations seem to work:

DT[, abs(C)]
# [1] 2.236068 2.236068 2.236068 2.236068

Though not always...

DT[ , sum(C), by = I]

Error in gsum(C) :
Type complex not supported by GForce sum (gsum). Either add the prefix base::sum(.) or turn off GForce optimization using options(datatable.optimize=1)

But

DT[ , base::sum(C), by = I]

Error in``[.data.table(DT, , base::sum(C), by = I) :
Type 15 in `.SD` column 0

@jangorecki
Copy link
Member

The last one DT[ , base::sum(C), by = I] looks like a bug. This sapply(split(DT$C, DT$I), sum) works.

@MichaelChirico MichaelChirico mentioned this issue Aug 3, 2016
Closed
@arunsrinivasan arunsrinivasan added this to the v2.0.0 milestone Aug 3, 2016
@arunsrinivasan
Copy link
Member Author

dup of #1444.

@mattdowle mattdowle removed this from the v1.9.10 milestone Nov 23, 2016
@eliocamp eliocamp mentioned this issue Jul 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mattdowle @arunsrinivasan @jangorecki @MichaelChirico
0