Updates to mediation_designer

[jaspercooper]

• simplify code
• add mediation analysis

[jaspercooper]

OK this is being pushed off to next big version.

One issue is the slowness induced by quasi-bayesian simulations. @macartan suggests adding an option to turn off estimators, which I think is a good solution.

also we should really consider submitting PRs to mediation and broom so that we can tidy() the mediation output.

• PR tidy method for mediate
• PR to broom
• clean up code and get looking good
• add option to switch off mediate()

[jaspercooper]

Where are we at with this, @Medinali ? Did we ever end up doing the PRs to broom and mediate? I believe all the other changes have been implemented, oder?

[lilymedina]

na klar!

• The PR to broom was merged, which is enough to be able to tidy(). Though I forgot to include CIs in tibble and also noticed that columns are of type character and not numeric, so will do a new PR to include those edits.

• Included estimands for indirect effects.

• The code is looking good but would look cleaner once the new PR has been merged.

• I implemented new args names in the code-- I really like how they make formals very intuitive but am a bit unsure about how they look on the printed code; there are lot of Z_s, Y_s and M_s.

remaining task

• finish up tibble and PR broom

• Run tests and checks and PR master

[jaspercooper]

That sounds amazing, thanks so much!