Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix R installation instructions (windows / mac) #15087

Merged
merged 2 commits into from
May 31, 2019

Conversation

aaronmarkham
Copy link
Contributor

Description

Shows a warning about R v3.6 and provides links to installation archives for older versions of R.

This is a workaround for:

We should leave those open as a reminder that the community would like R v3.6 support.

@aaronmarkham aaronmarkham requested a review from szha as a code owner May 28, 2019 22:36
@aaronmarkham
Copy link
Contributor Author

@hetong007 - please review.

Install the latest version (3.5.1+) of R from [CRAN](https://cran.r-project.org/bin/macosx/).
Note: packages for 3.6.x are not yet available.

Install 3.5.x of R from [CRAN](https://cran.r-project.org/bin/macosx/). The latest is [v3.5.3](https://cran.r-project.org/bin/macosx/R-3.6.0.pkg).
Copy link
Contributor

Choose a reason for hiding this comment

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

please replace the link with https://cran.r-project.org/bin/macosx/R-3.5.3.pkg

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops... good catch

@abhinavs95
Copy link
Contributor

@mxnet-label-bot add [Doc, R]

@aaronmarkham
Copy link
Contributor Author

Rebased due to error in CI. Reported here: #15095

Copy link
Contributor

@piyushghai piyushghai left a comment

Choose a reason for hiding this comment

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

LGTM.

@aaronmarkham aaronmarkham merged commit 866ec10 into apache:master May 31, 2019
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
* show warning about R v3.6

* fix link to R install
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants