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

Failure with dev tibble? #68

Open
krlmlr opened this issue Jul 22, 2021 · 4 comments
Open

Failure with dev tibble? #68

krlmlr opened this issue Jul 22, 2021 · 4 comments

Comments

@krlmlr
Copy link

krlmlr commented Jul 22, 2021

When trying to release tibble to CRAN, I received the following change to worse with the codebook package that could be related to one of the changes (now using pillar::dim_desc() instead of home-grown version). Could you please take a look? Is this a problem on tibble's end?

 --- re-building �codebook_qualtrics.Rmd� using rmarkdown

 Attaching package: 'dplyr'

 The following objects are masked from 'package:stats':

     filter, lag

 The following objects are masked from 'package:base':

     intersect, setdiff, setequal, union

 Quitting from lines 103-105 (codebook_qualtrics.Rmd) 
 Error: processing vignette 'codebook_qualtrics.Rmd' failed with diagnostics:
 Could not summarise item Q7. Error in `[<-.data.frame`(`*tmp*`, , j, value = c("# A tibble: 7 � 2", : replacement has 10 rows, data has 7
@krlmlr
Copy link
Author

krlmlr commented Jul 22, 2021

@rubenarslan: It seems that the error is in the last chunk in _codebook_item.Rmd . You can no longer rely on tibble::enframe() to produce an error if the input is a data frame. A simple fix consists of explicitly checking if the input is already a data frame.

@rubenarslan
Copy link
Owner

rubenarslan commented Jul 22, 2021

Thanks for already looking into this! This version is released on CRAN already? I'm about to go on holiday, can't manage a fix before then, so I guess there'll be some breakage for a while.

@krlmlr
Copy link
Author

krlmlr commented Jul 23, 2021

I'm not even sure enframe() should be supported for data frames. Let's tackle this when you're back, enjoy vacation!

@krlmlr
Copy link
Author

krlmlr commented Sep 26, 2021

Can you please install tidyverse/tibble#915 and let me know if codebook still fails its checks, and if so if is a tibble or a codebook problem?

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

2 participants