Skip to content

doc: document R CRAN mirror process#10211

Closed
lholmquist wants to merge 1 commit intonodejs:masterfrom
lholmquist:benchmark-doc-r-10204
Closed

doc: document R CRAN mirror process#10211
lholmquist wants to merge 1 commit intonodejs:masterfrom
lholmquist:benchmark-doc-r-10204

Conversation

@lholmquist
Copy link
Contributor

@lholmquist lholmquist commented Dec 10, 2016

Checklist
  • [x ] documentation is changed or added
  • [ x] commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

The current benchmark docs showing how to install R packages
doesn't go into how to specify a CRAN mirror.

This PR adds 3 options on how to install the needed packages
when a CRAN mirror is asked for.

fixes: #10204

@nodejs-github-bot nodejs-github-bot added dont-land-on-v7.x benchmark Issues and PRs related to the benchmark subsystem. labels Dec 10, 2016
@lholmquist lholmquist mentioned this pull request Dec 10, 2016
Closed
@AndreasMadsen
Copy link
Member

AndreasMadsen commented Dec 10, 2016

I would prefer just a single safe option, I think half a page of install instructions is kinda scary for the reader.

$ R
options(menu.graphics = FALSE)
install.packages("ggplot2")
install.packages("plyr")

or

$ R
install.packages("ggplot2", repo="http://cran.us.r-project.org")
install.packages("plyr", repo="http://cran.us.r-project.org")

should be sufficient.

@lholmquist
Copy link
Contributor Author

lholmquist commented Dec 12, 2016

@AndreasMadsen yea, that makes sense, i'll probably use the second option

@lholmquist lholmquist force-pushed the benchmark-doc-r-10204 branch from 6283b02 to acb7b7c Compare December 12, 2016 14:02
@lholmquist
Copy link
Contributor Author

lholmquist commented Dec 12, 2016

@AndreasMadsen ok, updated and rebased

@AndreasMadsen
Copy link
Member

AndreasMadsen commented Dec 12, 2016

LGTM. To be clear you should get that message on all platforms, but some may show it as an X11 window, which is what causes the problems.

@lholmquist
Copy link
Contributor Author

lholmquist commented Dec 12, 2016

@AndreasMadsen should i change the heading then? or leave as is?

@AndreasMadsen AndreasMadsen added the doc Issues and PRs related to the documentations. label Dec 12, 2016
@AndreasMadsen
Copy link
Member

AndreasMadsen commented Dec 12, 2016

@lholmquist You could change it to CRAN mirror or something similar.

@lholmquist lholmquist force-pushed the benchmark-doc-r-10204 branch from acb7b7c to 7af0970 Compare December 12, 2016 16:58
@lholmquist
Copy link
Contributor Author

lholmquist commented Dec 12, 2016

@AndreasMadsen ok, updated and rebased

lance
lance reviewed Dec 13, 2016
View reviewed changes
benchmark/README.md Outdated
Copy link
Member

@lance lance Dec 13, 2016

Choose a reason for hiding this comment

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

Typo: 'suits your location'

Copy link
Contributor Author

@lholmquist lholmquist Dec 13, 2016

Choose a reason for hiding this comment

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

updated

@lholmquist
doc: document R CRAN mirror process
fa5643c 
The current benchmark docs showing how to install R packages
doesn't go into how to specify a CRAN mirror.

This commit shows how to specify the repo parameter when
installing a package.

fixes: nodejs#10204
@lholmquist lholmquist force-pushed the benchmark-doc-r-10204 branch from 7af0970 to fa5643c Compare December 13, 2016 14:54
@lance
Copy link
Member

lance commented Dec 15, 2016

Landed in: eceef76

@lance lance closed this Dec 15, 2016
lance pushed a commit that referenced this pull request Dec 15, 2016
@lholmquist @lance
doc: document R CRAN mirror process
eceef76 
The current benchmark docs showing how to install R packages
doesn't go into how to specify a CRAN mirror.

This commit shows how to specify the repo parameter when
installing a package.

Fixes: #10204
PR-URL: #10211
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
italoacasas pushed a commit that referenced this pull request Dec 17, 2016
@lholmquist
doc: document R CRAN mirror process
6b98906 
The current benchmark docs showing how to install R packages
doesn't go into how to specify a CRAN mirror.

This commit shows how to specify the repo parameter when
installing a package.

Fixes: #10204
PR-URL: #10211
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
@italoacasas italoacasas mentioned this pull request Dec 17, 2016
Merged
cjihrig pushed a commit that referenced this pull request Dec 20, 2016
@lholmquist @cjihrig
doc: document R CRAN mirror process
6ca5e5f 
The current benchmark docs showing how to install R packages
doesn't go into how to specify a CRAN mirror.

This commit shows how to specify the repo parameter when
installing a package.

Fixes: #10204
PR-URL: #10211
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
@MylesBorins
Copy link
Contributor

MylesBorins commented Jan 22, 2017

This does not land cleanly in LTS. Added dont-land label. Please feel free to manually backport

@AndreasMadsen
Copy link
Member

AndreasMadsen commented Jan 23, 2017

It should not be backported, the entire benchmark suite was rewritten in v7 and declared a major change.

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

Reviewers

2 more reviewers

@lance lance lance left review comments

@AndreasMadsen AndreasMadsen AndreasMadsen approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

benchmark Issues and PRs related to the benchmark subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Benchmark docs: how to select the CRAN Mirror for R

5 participants

@lholmquist @AndreasMadsen @lance @MylesBorins @nodejs-github-bot