-
Notifications
You must be signed in to change notification settings - Fork 80
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
Html output in the R viewer (i.e. Better integration with the gt package) #667
Comments
YEs totally agree with this. gt is really a nice model. |
We're pretty close already with this sort of code snippet:
Four wants:
|
More iteration on this idea, but I think I've got it now. I think this is really getting us towards We need to create a new option: whether or not someone want to use rich html output (and make sure this aligns with Next we need to introduce a new concept to These functions will be generic. You should be able to call them on a complete skim_df, a skim_list (what you get from partition) and one_skim_df (the contents from partition). Otherwise, the last bit of this will be to clean up the summary. Right now, the summary generates the object and does some of the printing in the same step. We'll need to handle that in separate functions, so that we'll have more options for summary printing. |
Another extension here is to produce the results as an html widget. Then, for example, we could put each table (summary + stats for each type) within a tabset. Supporting that means our option for output type shouldn't just be boolean, but support multiple text values. Closer to an enum. |
This sounds amazing. Maybe we should be declaring a skimr 3 roadmap?
Elin
…On Thu, Dec 30, 2021 at 1:24 AM Michael Quinn ***@***.***> wrote:
Another extension here is to produce the results as an html widget. Then,
for example, we could put each table (summary + stats for each type) within
a tabset.
https://dev.to/imiahazel/pure-html-css-tabs-2p60
Supporting that means our option for output type shouldn't just be
boolean, but support multiple text values. Closer to an enum.
—
Reply to this email directly, view it on GitHub
<#667 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFYI7KFQGQAN5ODBPEOL33UTP3JPANCNFSM5AW4JKZA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you commented.Message ID:
***@***.***>
|
I have a suggestion for 1.
I'd be interested to see more! On my side, I did further modifications to improve the display, such as pct formatting, and auto formatting on gt side. It is out of scope for skimr though.. gt_skim <- function(data, type) {
gt::gt(data, caption = type) |>
gt::fmt_auto(lg_num_pref = "suf") |>
gt::fmt_percent(
columns = any_of("complete_rate"),
drop_trailing_zeros = TRUE,
decimals = 1
)
} |
The gtsummary package shows some great ideas for producing nice tables with gt.
http://www.danieldsjoberg.com/gtsummary/
It seems like skimr should also be able to support gt methods that produce nice html tables. This should be similar to our support of other approaches for doc-ready tables.
The text was updated successfully, but these errors were encountered: