-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
introduce decorators for tm_g_pp_therapy
#1268
introduce decorators for tm_g_pp_therapy
#1268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks fine but the example has an error that prevents the module to display.
Warning: Error in : One or more variables in the dataframe have class 'difftime'. Please convert to factor or character.
when evaluating qenv code:
table <- rlistings::as_listing(table, key_cols = NULL, default_formatting = list(all = fmt_config(align = "left")))
This prevents the module to display the plot with the caption:
@llrs-roche this is what I see |
Can you confirm you
and then. before running the example did you run devtools::load_all("../teal.reporter")
devtools::load_all("../teal")
devtools::load_all(".") |
I can confirm that I run all the steps above and I still see those warnings/errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing to CRAN's version for rlistings worked. I'm not sure if this should be a separate issue, sooner or later teal* packages will be facing this
Yup it’s the rlistings issue. Have to use character or factor column instead. |
Alrighty, the |
Part of insightsengineering/teal#1371
Working Example