-
Notifications
You must be signed in to change notification settings - Fork 78
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
Unable to build with R 3.6.0 on Mac OS 10.13.6 #519
Comments
We are looking into this now - sorry about this (and thanks very much for telling us about it). |
Thanks @gvwilson! I'm tagging @ivelasq in on this, who's been guiding me through the process. Here's more info: I can reproduce the error when I
Ch 7 knits properly when I
|
Kudos to @ivelasq for teaming up on this. We're pretty sure this has to do with a missing Gill Sans MT font, at least on my Mac. @gvwilson do you have Gill Sans MT installed on your Mac? After I installed Gill Sans MT and ran
it worked fine. Then we restarted R and knitted Ch 7, which also worked. @ivelasq is looking for a way for {dataedu} to check for Gill Sans MT. Then we can default to a different font if Gill Sans MT is not installed. Worst case scenario is we'll just change the font to something we're confident most people will have. @gvwilson as a temporary solution, feel free to remove + Super curious about why a missing font would cause the polygon error though. Any ideas on that? |
@restrellado can you say from where you downloaded Gill Sans MT? Found one set of fonts to download, but none seem to be the "base" font (i.e., they're for bold, italic, etc.). I may be misunderstanding how font files work.Having what I think is this same issue with a fresh install of the pkg.! |
Hi @gvwilson, we've updated the {dataedu} package and hope that it resolves the font issue you were encountering. Please reinstall and let us know how it goes! Thanks! |
Hi @ivelasq - unfortunately it still fails to build for me. I have done the following to correct early errors:
but it still fails with:
I am sync'd with commit bf7cc9c. |
Hi @gvwilson Sorry that happened! This can happen if you don't have the NRC Word-Emotion Association Lexicon downloaded yet. If that's true for you, can you please try running this in your console:
If you don't already have NRC downloaded, R should prompt you to download it. I think the build breaks when it gets to this part because it doesn't have a way to get a "Yes" from you. Let us know if that works or if you've already tried that! |
Thanks - I installed that interactively and re-ran
The first two were not available for R 3.6.0 on the first CRAN mirror I tried, but were available from the Tokyo mirror. Even with these installed, the build failed because of a missing package, but didn't give me the name. The build log is below. Thanks,
|
Hi @gvwilson ! I think we're almost there! I suspect that the missing package is the {ranger} package - could you install and let us know how it goes? Thanks! |
Hi @ivelasq - |
Hi @gvwilson , could you please try with the {randomForest} package installed whenever you get the chance? Another thought is that maybe the {ranger} library needs to be called in the RMarkdown file - that wasn't the case when we rendered it in the past, though. Thanks! |
I have installed
|
Made some more progress but am still blocked:
I edited
|
Hi Greg! Going to try a different approach–I'll do a fresh clone and see if I can reproduce it too. More soon! |
Hi again! Ok so we were able to get it to render after downloading a fresh clone of the repo. The only changes we had to make were installing packages I didn't have. We used
@ivelasq did a little more research and thinks it might be a missing Arial Narrow font, based on another similar issue she found. I'll let Isabella explain the thing she thought you could try. Take it away @ivelasq! |
Thanks Ryan! Hi Greg, thanks for the update. Like Ryan said, I found this issue where it states that Arial Narrow was missing, which was causing the problem. They download it using {extrafont}. Could you please try that, and let us know if it works? I actually hadn't noticed, but I was missing this font as well! For some reason, my render still went through but it did give me a bunch of warnings: https://datascienceineducation.com/c12.html#plotting-the-network Greg, one more note - it seems like previously, you were able to get past chapter 12 (as the {e1701} installation fixed something in chapter 14). If there's anything that has changed since then that could help us figure out what is happening in chapter 12, please let us know! Thanks! |
Thanks for the feedback - here's where I've gotten to: Re-run `devtools::install_github('data-edu/dataedu')Note: maybe add a section in Chapter 2 on installation instructions, either standalone or a forward ref to Section 5.7? O'Reilly books have conditioned me to look for installation instructions early on, and I think other instructors might as well.
|
Thanks Greg! Closing issue now. |
HEAD
is5df2127ff
bash ./_build.sh
produces warnings about missing packagesInstallation:
R:
Output from
_build.sh
:The text was updated successfully, but these errors were encountered: