-
Notifications
You must be signed in to change notification settings - Fork 82
Description
I noticed a few issues with the t-test vignette: https://infer.netlify.app/articles/t_test
-
The first paragraph states "We’ll start out with a 1-sample t-test, which compares a sample mean to a hypothesized true mean value. Then, we’ll discuss paired t-tests, which are a special use case of 1-sample t-tests, and evaluate whether differences in paired values (e.g. some measure taken of a person before and after an experiment) differ from 0." However, discussion of paired t-tests is not included later in the vignette.
-
The 2-sample t-test section includes the following statement: "Again, note the warning about missing values—many respondents’ values are missing. If we were actually carrying out this hypothesis test, we might look further into how this data was collected; it’s possible that whether or not a value in either of these columns is missing is related to what that value would be." It's not clear what this is referring to as there is no prior output referring to missing values. Furthermore when I inspect the dataset used in the vignette (gss) I don't find any rows with missing values.
-
Also in the 2-sample t-test section the following phrase can be found: "theoretical p-values are not yet supported". This seems to be a mistake as theoretical p-values are calculated later in the vignette via the t_test() function.