-
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
A few remarks #121
Comments
Thanks @Degoot-AM, will look into these. |
Had a meeting with Degoot today and was able to go over his points above. Most of the issues I was able to explain to him but
Given the interventions...
This function signature is more intuitive...
Combining the models should be internal and hidden from the user. I will submit a PR for 6. for you to review. |
Hi @bahadzie, thanks for looking into this. Could you please hold off on these. Re: 1. I'm re-doing the vignette sections relating to the infection class, so I can take this up there. Re: 6. The function signature as it is needs a named list to be passed to the |
The text was updated successfully, but these errors were encountered: