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

Suggestion: import Orthodont data #321

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

seaCatKim
Copy link

As written, the text seems to imply that the Orthodont data is from the lme4 package. Suggest moving and visibly reading in the data from the nlme package before the first linear model in section 7.4.1 to avoid confusion.

As written, the text seems to imply that the Orthodont data is from the lme4 package. Suggest adding in the line reading in the data from the nlme package before the first linear model and including it in the text to avoid confusion.
@seaCatKim
Copy link
Author

Similar suggestion for the survival model later in the same section - not clear where the ovarian data set comes from. Seems to come from the survival package: data(cancer, package = "survival")?

@juliasilge
Copy link
Member

Thanks for this PR @seaCatKim! 🙌 We are in the process of final printing for the first edition of this book but will look at this after we get that edition off and finished.

@presnell
Copy link

presnell commented Aug 5, 2022

Came to make the same comment: data(Orthodont, package = "nlme") is needed before using the Orthodont dataset in section 7.4.1 (Special formulas and inline functions). The ovarian dataset is automatically loaded with the survival package, so you're arguably ok there since you already have library(censored) which loads the survival package.

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

Successfully merging this pull request may close these issues.

3 participants