survey_prop should default to proportions

Open szimmer opened this issue 3 years ago • 2 comments

This seems like a more logical default given the context people will expect. Documentation already states:

This addresses issue #141

Mar 11 '22 16:03 szimmer

Codecov Report

Merging #142 (dfef559) into main (ac3fb35) will decrease coverage by 0.07%. The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main     #142      +/-   ##
==========================================
- Coverage   83.27%   83.19%   -0.08%     
==========================================
  Files          22       22              
  Lines        1106     1113       +7     
==========================================
+ Hits          921      926       +5     
- Misses        185      187       +2

Impacted Files	Coverage Δ
R/survey_statistics.r	`86.98% <80.00%> (-0.68%)`	:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ac3fb35...dfef559. Read the comment docs.

Mar 23 '22 00:03 codecov-commenter

Updated with messaging that default behavior has changed. While I was at it, I added another method to estimate confidence intervals.

Mar 23 '22 01:03 szimmer