-
Notifications
You must be signed in to change notification settings - Fork 7
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
Ebola vignette clean-up #290
Comments
@seabbs that's fair regarding this being 6 issues vs 1 issue. I don't have enough personal experience to have preference yet but I can see the appeal of 6 issues. It was easier for me initially to do 1 issue and it felt like "I just want to get this gone" hence where we are. |
I think I would track this with a meta-issue but assign each as individual issues to give the option of splitting into their own PRs |
@seabbs can we leave this as it is for the time being? We are likely making changes to the formula interface anyway soon. |
The Ebola vignette PR (#52) got really big. There are a few comments left to go over but I recommend we merge a rougher version of the vignette, then I'll pick these up in a new issue. One reason for that is that I'm finding it a bit unweildy with such a large comment history.
bf(mu ~ 1, sigma ~ 1)
or just leave as default. I prefer writing it out, Sam prefers leaving it as default.ptime
andptime_daily
. See Refactor ofobserve_process
#291 also. (Moved to Extensions for Ebola vignette #285)::
in text the first time a function is introduced and then not using::
in code. Also using()
to get automaticpkgdown
linksOther things I notice
head(obs_prep)
twicesex = case_when
over and over -- way to fix this?The text was updated successfully, but these errors were encountered: