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

Add requisite libraries for ArrayFire #135

Merged
merged 4 commits into from
Jul 19, 2016
Merged

Conversation

ranjanan
Copy link
Contributor

ArrayFire fails on PackageEvaluator. This should make it work.

@tkelman
Copy link
Contributor

tkelman commented Jul 15, 2016

can't the package be made to handle installation automatically?

@ranjanan
Copy link
Contributor Author

You need a login and a password to download the binary. I'm pulling from their Jenkins CI with persmission.

@@ -89,6 +89,13 @@ export PYTHON=""
sudo apt-get install wamerican
# Need xmllint (and others?) for XMLDict.jl
sudo apt-get install libxml2-utils
# ArrayFire
# ArrayFire
Copy link
Contributor

Choose a reason for hiding this comment

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

don't duplicate this

@tkelman
Copy link
Contributor

tkelman commented Jul 15, 2016

ah, ok. why are they using atlas?

@ranjanan
Copy link
Contributor Author

It is listed as a pre-requisite in their docs. I think this is because it doesn't ship with a BLAS, so it tries to link to system BLAS by default (and that step is just in case a system BLAS isn't installed). You can build by linking to OpenBLAS, but I haven't tried linking the binary with libopenblas.so

@tkelman
Copy link
Contributor

tkelman commented Jul 16, 2016

Can you test if it works with openblas instead? Would rather not install things we don't need. And why do you need a login to download a binary, isn't arrayfire open source now?

@ranjanan
Copy link
Contributor Author

Yes it is, in that you can clone the library and build it from source. That's going to take time though. And the reason I don't automate building from source is that there are many ways you would do that depending on the hardware you have.

@ranjanan
Copy link
Contributor Author

For a minimum install, it turns out you don't need any of those libraries, so I'm removing ATLAS, FreeImage, etc.

@ranjanan
Copy link
Contributor Author

Fails because of this issue

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2016

ArrayFire is really more than 650 MB? Why is it such a large download?

@ranjanan
Copy link
Contributor Author

They're bundling a number of libraries with it. Like CLBLAS, CLFFT, etc.

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2016

We won't be testing any of those on PkgEval, will we? They don't have a smaller CPU-only version?

@ranjanan
Copy link
Contributor Author

No, they don't have anything lighter than this.

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2016

I guess we can try this, but adding 650+ MB (that's just the download, then duplicated or more once installed?) to each worker VM might be a bit much. It's an open-source library, I think the best path forward is going to be just to build a CPU-only binary ourselves for testing purposes.

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2016

Do you know if the shell script deletes itself after it's done?

@ranjanan
Copy link
Contributor Author

No it doesn't. Should I throw in a command to delete it after extraction?

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2016

Yeah, I think that would be good.

@tkelman tkelman merged commit cac4592 into JuliaCI:master Jul 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants