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

json.column error after upgrading to dplyr 0.7.0 #58

Open
MarkEdmondson1234 opened this issue Jun 13, 2017 · 4 comments
Open

json.column error after upgrading to dplyr 0.7.0 #58

MarkEdmondson1234 opened this issue Jun 13, 2017 · 4 comments

Comments

@MarkEdmondson1234
Copy link

MarkEdmondson1234 commented Jun 13, 2017

Hi there, I use tidyjson in the googleAnalyticsR package, and this week people have reported an error in a function that uses tidyjson. It turns out that those people who are using dplyr version 0.7.0 get the error:

https://github.com/MarkEdmondson1234/googleAnalyticsR/issues/87

For now a fix is to downgrade dplyr via devtools::install_version("dplyr", version = "0.5.0") but that is obviously not permanent :)

@colearendt
Copy link

I will look through to be sure the issue you are seeing is the one we anticipated. Please see issue here. We are looking to test and release a fix to CRAN as soon as possible.

The problem we have seen is with the dplyr verbs, as they are no longer calling the NSE versions behind the scenes. Although a less than ideal workaround, using mutate_, for instance, should solve your issues, as those are properly wrapped by the legacy version of tidyjson.

@colearendt
Copy link

Also, as a random aside - I like to use packrat to protect my work from version issues. I always feel a necessity to plug when version issues arise, because they can be a bit of a nightmare.

@gaiusjaugustus
Copy link

gaiusjaugustus commented Dec 11, 2017

I'm having the same issue. I've downgraded to dplyr 0.5.0, but am still getting this error. Any other advice or updates on this?

Error in eval(assertion, env) : 
  argument "json.column" is missing, with no default
sessionInfo()
R version 3.4.3 (2017-11-30)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 7 x64 (build 7601) Service Pack 1

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252 
[2] LC_CTYPE=English_United States.1252   
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C                          
[5] LC_TIME=English_United States.1252    

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] jsonlite_1.5   tidyjson_0.2.1 dplyr_0.5.0   

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.14     digest_0.6.12    withr_2.1.0      assertthat_0.2.0
 [5] R6_2.2.2         DBI_0.7          magrittr_1.5     httr_1.3.1      
 [9] rlang_0.1.4      curl_3.0         devtools_1.13.4  tools_3.4.3     
[13] yaml_2.1.15      compiler_3.4.3   memoise_1.1.0    knitr_1.17      
[17] tibble_1.3.4  

@colearendt
Copy link

colearendt commented Dec 14, 2017

@gaiusjaugustus Any clarification on what your pipeline looks like? Can you provide a reprex? This issue is not uncommon since dplyr 0.7.0 came out, and unfortunately has many causes.

Also, this repo is deader than jeremystan/tidyjson (which is having troubles of its own). Probably best to continue discussion here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants