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

Switch back to using OpenBLAS for Ubuntu 22 #122

Merged
merged 1 commit into from
Apr 29, 2022
Merged

Conversation

glin
Copy link
Contributor

@glin glin commented Apr 28, 2022

Switches the Ubuntu 22 R builds back to using OpenBLAS. When we added Ubuntu 20 support, there was a bug with the OpenBLAS package that caused hangs/crashes with R, so we replaced it with the (slower) default reference BLAS (#62). That bug doesn't affect Ubuntu 22 and was probably patched on Ubuntu 20 already, so we can switch back to using OpenBLAS. This change essentially reverts 1bedbc6 but for Ubuntu 22 (which was mostly copied from Ubuntu 20)

This change shouldn't break any existing installations of R on Ubuntu 22. You'll be able to reinstall R and continue using any previously installed packages. Ubuntu/Debian makes BLAS libraries swappable at runtime with the alternatives system, so you could also make this change on an existing installation by running apt install libopenblas-dev, and R will automatically switch to using OpenBLAS. You can also revert back to using the default BLAS with the update-alternatives command if you wanted to.

Ubuntu 20 builds were switched to the default reference BLAS because
of a bug with the OpenBLAS package. That bug does not seem to affect
Ubuntu 22, so switch back to using OpenBLAS for Ubuntu 22, for
consistency with all other R builds.
@glin glin requested a review from a team April 28, 2022 23:39
@glin glin merged commit 88992af into master Apr 29, 2022
@glin glin deleted the glin/jammy-openblas branch April 29, 2022 16:36
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