1445: Add cmh_sato method to estimate_proportion_diff()

[danielinteractive]

closes #1445

[danielinteractive]

I have read the CLA Document and I hereby sign the CLA

[Melkiades]

Just a question. Otherwise looks very good!! Thanks. I will run revdeps too ;)

[danielinteractive]

Thanks @Melkiades , I see some unrelated check failures, cannot do much about those

[Melkiades]

Lgtm!! Thanks @danielinteractive! Issues stem from recent merging of round_type from J&J. Merge of rtables'PRs should fix these. Maybe we can merge this anyway ^^

[gmbecker]

upstream pr has been merged and I've confirmed that the vignettes build for me locally with the latest formatters and rtables (which is what was failing in the GH action). Assumedly this is a cache issue and will clear once it gets the right versions of things. I'll leave the rest to you @Melkiades

[danielinteractive]

Thanks @gmbecker ! I just retriggered checks, they failed again but I guess the new version of rtables has not arrived here for some reason?

[shajoezhu]

hi Daniel, can you add the following lines to the DESCRIPTION

Remotes:
    insightsengineering/formatters@main,
    insightsengineering/rtables@main

this will help to get the right versions

[shajoezhu]

also need to bump the minimal version for formatters and rtables in the description