efficientR icon indicating copy to clipboard operation
efficientR copied to clipboard

Suspected typo in Chapter 1

Open michaelosheaa opened this issue 4 years ago • 6 comments

I noticed in Line 76 of Chapter 1 the sentence: "It is well known that R code can promote algorithmic efficiency compared with low level languages for certain tasks, especially if the code was written by someone who doesn't fully understand the language."

  • Should this instead be "algorithmic inefficiency"?

michaelosheaa avatar Jun 13 '21 09:06 michaelosheaa

Hi @michaelosheaa yes I think that is a typo! Fix would be most welcome, thanks for pointing this out.

Robinlovelace avatar Jul 15 '21 06:07 Robinlovelace

@Robinlovelace , can I make a pull request to fix this typo 😄?

shafayetShafee avatar Jun 14 '22 16:06 shafayetShafee

Hi @shafayetShafee yes of course, that would be most welcome. Many thanks!

Robinlovelace avatar Jun 14 '22 17:06 Robinlovelace

@Robinlovelace, hey I am noticing that this typo still exists in the online version of the book, although I have corrected it and PR got merged. Did I miss something? Sorry if I did something wrong 😓!

shafayetShafee avatar Jun 17 '22 08:06 shafayetShafee

No that is probably an issue with the build process. Can you help @csgillespie ?

Robinlovelace avatar Jun 17 '22 08:06 Robinlovelace

This is going to be tricky. It was originally set up via .travis many, many years ago. Converting to GitHub Actions is going to take a little time.

csgillespie avatar Jun 17 '22 09:06 csgillespie