-
Notifications
You must be signed in to change notification settings - Fork 704
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
Changes to support building R-keras and dependencies on POWER9 #9544
Changes to support building R-keras and dependencies on POWER9 #9544
Conversation
Patch for R base 3.6.2 from https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=946836 |
I took the config.guess from the easybuild ConfigureMake sources |
Travis test report: 2/2 runs failed - see https://travis-ci.org/easybuilders/easybuild-easyconfigs/builds/627723995 Only showing partial log for 1st failed test suite run 18994.1;
*bleep, bloop, I'm just a bot (boegelbot v20180813.01)*Please talk to my owner @boegel if you notice you me acting stupid),or submit a pull request to https://github.com/boegel/boegelbot fix the problem. |
R builds on POWER9 now, with the exception of extensions OpenCL and gpuR - which require the OpenCL headers and library. On Intel they ship in CUDA, but the CUDA ppc64le .run file doesn't include OpenCL. I'm talking to IBM about where we should get OpenCL on ppc64le... |
cdeterman/gpuR#81 suggest to use pocl (http://portablecl.org/) on ppc64le. So updating https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/pocl/pocl-1.3-GCC-8.2.0-2.31.1.eb may take us forward. |
@edmondac By "Intel" do you mean actual Intel as in not even on AMD processors, or just regular old boring x86-64? It's hard to tell at a glance which one you is meant when skimming through the quite rapid feed of EasyBuild mails. |
@edmondac #9593 adds pocl 1.4. I've done a test with the CUDA version and it allows the gpuR and OpenCL extensions to install. The log shows the following at the end of installing gpuR, which is certainly encouraging:
|
Sorry, I mean x86-64 |
Travis test report: 2/2 runs failed - see https://travis-ci.org/easybuilders/easybuild-easyconfigs/builds/633212341 Only showing partial log for 1st failed test suite run 19162.1;
*bleep, bloop, I'm just a bot (boegelbot v20180813.01)*Please talk to my owner @boegel if you notice you me acting stupid),or submit a pull request to https://github.com/boegel/boegelbot fix the problem. |
Don't need this patch if we merge easybuilders/easybuild-easyblocks#1938
Don't need this patch if we merge easybuilders/easybuild-easyblocks#1938
Don't need this patch if we merge easybuilders/easybuild-easyblocks#1938
Travis test report: 2/2 runs failed - see https://travis-ci.org/easybuilders/easybuild-easyconfigs/builds/640812406 Only showing partial log for 1st failed test suite run 19572.1;
*bleep, bloop, I'm just a bot (boegelbot v20180813.01)*Please talk to my owner @boegel if you notice you me acting stupid),or submit a pull request to https://github.com/boegel/boegelbot fix the problem. |
As this seemingly went stale I extracted the fix for the long double issue for R 3.6.2 into #9830 which also reduces the amount of changes |
From @Flamefire on #9538
|
It's not stale - there are just several things that need to be merged first: easybuilders/easybuild-easyblocks#1892 then #9764 AND |
Ah ok, they weren't mentioned initially (I guess?) and there was WIP. Anyway I'd suggest to continue with #9830 as the patch has better description and the unrelated changes are already removed. |
OK - so all that is left on here are the changes to R for ppc64le. easybuilders/easybuild-easyblocks#1949 and #9830 will cover that - so I'll close this PR. |
(created using
eb --new-pr
)This requires: