-
Notifications
You must be signed in to change notification settings - Fork 2
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 run_pipeline()
first draft
#149
base: main
Are you sure you want to change the base?
Conversation
Since this is an implementation of his design
@Bisaloo - this looks good to me. Below is what I think about the points you raised:
I suggest to compare it with the corresponding template in the package and:
I prefer the data object due to the following:
Maybe provide more clarification here if possible. Are you referring to a scenario where severity and transmissibility is executed? or where transmissibility is executed with the different methods? |
Yes, this. The |
Then I think one report with multiple tabs will be better. {reactable} allows for this but I have not yet figured out how not to show the severity tab (which will be empty) if the severity template is not executed. |
This should be achievable through rmdscaffold. Can it wait until episoap 0.2.0? |
NEWS.md
This is a first proposal for a simple
run_pipeline()
function. We can iterate on it as long as necessary, with the constraint that we won't touch the reports, or write any epi code here.Specific points to discuss:
out_dir
Could also be used to solve #51, #88.
TODO: