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

[hail] A compatible BLAS library is an install requirement #8647

Merged
merged 2 commits into from
Apr 29, 2020

Conversation

danking
Copy link
Contributor

@danking danking commented Apr 27, 2020

It's clear we are failing to explain the necessity of a compatible BLAS library. I hope this will stem the flow of bug tickets and support questions related to a misconfigured, missing, or incompatible BLAS library.

- lz4
- a C++ compiler,
- lz4,
- a compatible BLAS installation such as the Intel MKL or OpenBLAS
Copy link
Contributor

Choose a reason for hiding this comment

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

I might say "BLAS and LAPACK" in all of the places you're currently saying "BLAS".


and Mac OS X users, might try::

xcode-select --install
brew install lz4

Mac OS X already has compatible BLAS libraries.
Copy link
Contributor

Choose a reason for hiding this comment

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

See above

@@ -101,6 +103,9 @@ command builds Hail for Spark 2.4.0, installs the Python library, and installs
all the Python dependencies::

make install-on-cluster HAIL_COMPILE_NATIVES=1 SPARK_VERSION=2.4.0

Moreover, every worker node of the cluster needs a compatible BLAS library, such
Copy link
Contributor

Choose a reason for hiding this comment

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

See above

@danking
Copy link
Contributor Author

danking commented Apr 29, 2020

addressed

@danking danking merged commit 6befa05 into master Apr 29, 2020
jigold pushed a commit to jigold/hail that referenced this pull request Apr 30, 2020
)

* Update getting_started.rst

* address comments
@tpoterba tpoterba deleted the danking-patch-1 branch May 28, 2020 11:08
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