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

Refactor installation #476

Merged
merged 2 commits into from
Jul 27, 2021
Merged

Refactor installation #476

merged 2 commits into from
Jul 27, 2021

Conversation

t-kalinowski
Copy link
Member

This PR refactors install_tensorflow() and related CI tests.

Installation:

  • consolidated logic and documentation between keras::install_keras() and tensorflow::install_tensorflow(). install_keras() is now a mostly thin wrapper around install_tensorflow() that also pulls some additional python packages.
  • Added a documentation section about Apple Silicon support
  • If patch level is not supplied, install_tensorflow() will pull the latest available patch release (e.g., "2.4" now pulls "2.4.2", previously it would pull "2.4.0"

CI:

  • Removed the use of environment variables and optional function arguments as much as possible in the CI tests. The goal is for tests to closer match what will be the default in most situations.
  • Add testing for additional versions of R: oldrel, devel, and 3.6

'testthat' imports 'waldo' imports 'rematch2' imports 'tibble',
except that on github actions runner macOS-latest with r-release (4.1) tibble is not
installed as part of `install_local(dependencies=TRUE)`. This only presents on macOS with R 4.1.
Likely a transient bug not worth tracking down.

This refactor replaces `expect_that` with `expect`,
to avoid the need for tibble.
@t-kalinowski
Copy link
Member Author

t-kalinowski commented Jul 27, 2021

Hello,

The code that installs tensorflow and keras has been refactored.
Please update to the latest versions of reticulate, tensorflow, and keras:

install.packages("remotes")
remotes::install_github(paste0("rstudio/", c("reticulate", "tensorflow", "keras")))
reticulate::install_miniconda() # skip this if you want to self-install conda or use venv
keras::install_keras()

Please let us know if after updating you are still encountering installation issues:
https://github.com/rstudio/tensorflow/issues/new

closes #445
closes #465
closes #466
closes #438
closes #437 # windows
closes #436 # windows
closes #425
closes #407
closes #400
closes #370
closes #364

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