-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add teal.reporter
into tmh modules
#238
Comments
@nikolas-burkoff does it involve chunks business? or not? |
It does not |
very good :-) then yes please :-) |
It would involve adding
See the bit inside ### Reporter ### in this example app insightsengineering/teal#635 (comment) and also the example here https://github.com/insightsengineering/teal.reporter/blob/fba8c9b993e152a490556cf1e2a61ec285bd6122/vignettes/previewerReporter.Rmd#L136 (which is more complex as it has a switch statement to capture three different outputs) |
thx @nikolas-burkoff yeah if core team can drive this then I am happy to chat about what to record etc. |
teal.reporting
into tmh modulesteal.reporter
into tmh modules
@danielinteractive Could you list the specific encodings that are needed for every module to be included in the report? |
@mhallal1 maybe it is easiest just to include all? |
After add reporter teal#635 teal applications can be used to generate reports - do we want them added into tmh as well?
The text was updated successfully, but these errors were encountered: