Skip to content
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

Closed
12 tasks done
athowes opened this issue Sep 9, 2024 · 3 comments
Closed
12 tasks done

Ebola vignette clean-up #290

athowes opened this issue Sep 9, 2024 · 3 comments
Labels
high Required for next release

Comments

@athowes
Copy link
Collaborator

athowes commented Sep 9, 2024

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.

Other things I notice

  • Title has ` failing to render properly
  • Maths is failing to render properly e.g. (x \sim \mathcal{N}(\mu, \sigma)). (Moved to Make sure maths renders correctly in vignettes #333)
  • Calling head(obs_prep) twice
  • I don't like that I have to call sex = case_when over and over -- way to fix this?
  • Figure 2.3 panel C is too small / needs sorting
  • Figure 2.4 panel C also too small / needs sorting
@athowes
Copy link
Collaborator Author

athowes commented Sep 9, 2024

@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.

@seabbs
Copy link
Contributor

seabbs commented Sep 9, 2024

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

@athowes
Copy link
Collaborator Author

athowes commented Sep 13, 2024

Should we use bf(mu ~ 1, sigma ~ 1) or just leave as default. I prefer writing it out, Sam prefers leaving it as default.

@seabbs can we leave this as it is for the time being? We are likely making changes to the formula interface anyway soon.

@athowes athowes closed this as completed Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high Required for next release
Projects
None yet
Development

No branches or pull requests

2 participants