Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 06-stats-models-ci.qmd #316

Closed
wants to merge 1 commit into from
Closed

Conversation

LucyMcGowan
Copy link
Collaborator

Consider switching the number of users to uniform so you can't have negative numbers? I am resimulating this in class and adding in one where we have a non-linear relationship and plotting it so the negative number of users is a bit glaring.

Consider switching the number of users to uniform so you can't have negative numbers? I am resimulating this in class and adding in one where we have a non-linear relationship and plotting it so the negative number of users is a bit glaring.
@LucyMcGowan
Copy link
Collaborator Author

We also start by saying satisfaction is normal with a mean of 0 and SD 1 and then switch to it being some score (with averages around 70) - maybe just say it is some score? or pick one or the other and rescale?

@malcolmbarrett
Copy link
Collaborator

the failing state is unrelated to this change

these both sound reasonable to me

@malcolmbarrett
Copy link
Collaborator

If you run usethis::pr_merge_main() and push, this should pass (excepting any issues in this PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants