moderndive / ModernDive_book

Statistical Inference via Data Science: A ModernDive into R and the Tidyverse
https://www.moderndive.com/
Other
758 stars 486 forks source link

Potential improvements/checklist for v2/2nd edition #366

Open rudeboybert opened 4 years ago

rudeboybert commented 4 years ago

To be done for online edition of v2.1 or later

General

nycflights23

@avaldivi6

wmorgan485 commented 3 years ago

In Section 9.6.1, the discussion about the warning message to check that conditions have been met for the theoretical method {infer} might be rewritten for clarity. The warning message is generated by a visualize() command with method = "both". However, the discussion about the warning message follows a get_p_value() command using a simulation-based null distribution (null_distribution_movies_t), where this warning doesn't apply (right?). It might be clearer to discuss the warning message before getting the p-value. Or get_p_value might be run instead with a null distribution generated using assume(distribution = "t"), where the warning message would then apply.