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

unclear error from pagination with colwidhts #110

Closed
Melkiades opened this issue Apr 11, 2023 · 2 comments
Closed

unclear error from pagination with colwidhts #110

Melkiades opened this issue Apr 11, 2023 · 2 comments
Labels
bug Something isn't working sme

Comments

@Melkiades
Copy link
Contributor

library(rlistings)
library(dplyr)

anl <- ex_adsl
anl <- anl[1:10, c("USUBJID", "ARM", "BMRKR1")]
anl <- var_relabel(anl,
  USUBJID = "Unique\nSubject\nIdentifier",
  ARM = "Description\nOf\nPlanned Arm"
)
lsting <- as_listing(ex_adae[1:25, ], disp_cols =  c("USUBJID", "AESOC", "RACE")
>   pag <- paginate_listing(lsting, colwidths = c(15, 15, 15, 15))
Error in if (nc[currentIndex] == 0L) upperBlockIndex <- c(upperBlockIndex, :
missing value where TRUE/FALSE needed

We need to check what was left out of the pagination machinery in paginate_listing from paginate_table

@Melkiades Melkiades added bug Something isn't working sme labels Apr 11, 2023
@gmbecker
Copy link
Contributor

Right, so this came from the fact that the listings machinery was (incorrectly, I still assert) was using the names of colwidths. So I suppose the assertion was correct, but should not have been needed. I will create a PR that fixes this, as I already tracked it down.

@gmbecker
Copy link
Contributor

Closed by #109

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working sme
Projects
None yet
Development

No branches or pull requests

2 participants