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.

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

top_n crashes #2279

Closed
burkovae opened this issue Dec 1, 2016 · 1 comment
Closed

top_n crashes #2279

burkovae opened this issue Dec 1, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@burkovae
Copy link

@burkovae burkovae commented Dec 1, 2016

Recently I was experimenting with top_n and this (wrong) application of top_n does fry all R sessions:

data.frame(x = 1:10) %>% top_n(10, "x")

 *** caught segfault ***
address 0x20000010, cause 'memory not mapped'

Traceback:
 1: .Call("dplyr_filter_impl", PACKAGE = "dplyr", df, dots)
 2: filter_impl(.data, dots)                                                                                                                                                                     
 3: filter_.tbl_df(tbl_df(.data), .dots = dots)                                                                                                                                                  
 4: filter_(tbl_df(.data), .dots = dots)                                                                                                                                                         
 5: as.data.frame(filter_(tbl_df(.data), .dots = dots))                                                                                                                                          
 6: filter_.data.frame(.data, .dots = lazyeval::lazy_dots(...))                                                                                                                                  
 7: filter_(.data, .dots = lazyeval::lazy_dots(...))                                                                                                                                             
 8: filter(x, min_rank(desc("x")) <= 10)                                                                                                                                                         
 9: eval(expr, envir, enclos)                                                                                                                                                                    
10: eval(call)                                                                                                                                                                                   
11: top_n(., 10, "x")                                                                                                                                                                            
12: function_list[[k]](value)                                                                                                                                                                    
13: withVisible(function_list[[k]](value))                                                                                                                                                       
14: freduce(value, `_function_list`)                                                                                                                                                             
15: `_fseq`(`_lhs`)                                                                                                                                                                              
16: eval(expr, envir, enclos)
17: eval(quote(`_fseq`(`_lhs`)), env, env)
18: withVisible(eval(quote(`_fseq`(`_lhs`)), env, env))
19: data.frame(x = 1:10) %>% top_n(10, "x")
@krlmlr
Copy link
Member

@krlmlr krlmlr commented Dec 1, 2016

Thanks, confirmed.

Loading

@krlmlr krlmlr self-assigned this Feb 10, 2017
@krlmlr krlmlr added this to the data frame 1 milestone Feb 10, 2017
@krlmlr krlmlr added this to the data frame 1 milestone Feb 10, 2017
@krlmlr krlmlr closed this in #2444 Feb 20, 2017
krlmlr added a commit to krlmlr/dplyr that referenced this issue Feb 20, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Jun 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants