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

Add Chapter 14 notes #16

Merged
merged 14 commits into from
Apr 27, 2024
Merged

Add Chapter 14 notes #16

merged 14 commits into from
Apr 27, 2024

Conversation

rserran
Copy link
Contributor

@rserran rserran commented Apr 27, 2024

No description provided.

Copy link
Member

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Member

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an error in 15-construct-explainers:

Quitting from lines  at lines 2296-2311 [15-construct-explainers] (bookclub-ema.Rmd)
Error in `explain()`:
! argument "x" is missing, with no default
Backtrace:
 1. lime::explain(...)
Execution halted

Either make that work, or disable evaluation of that block.

You should always ctrl-shift-k to render the individual chapter on your own machine at a minimum, and ideally ctrl-shift-b to build the entire book and make sure nothing breaks. If I can't review this before your meeting, you should render them on your machine and run them from there.

@jonthegeek jonthegeek enabled auto-merge (squash) April 27, 2024 13:39
auto-merge was automatically disabled April 27, 2024 14:04

Head branch was pushed to by a user without write access

Copy link
Member

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jonthegeek jonthegeek merged commit add5fa9 into r4ds:main Apr 27, 2024
1 check passed
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