-
Notifications
You must be signed in to change notification settings - Fork 91
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
LC Spack Support #1204
LC Spack Support #1204
Conversation
@AntoineMazuyer I could use some help building VTK, specifically does this look like the right package to you? https://github.com/spack/spack/blob/develop/var/spack/repos/builtin/packages/vtk/package.py Basically the build options are the You can see that VTK has a huge list of hard dependencies, |
@corbett5, just a comment about two packages:
|
@AF1990 Adding support for newer versions of SuiteSparse is trivial, I just haven't done it yet. I can confirm though that it works with 5.7.2 (the latest version spack supports). As for PETSc I think I already built with 3.14, one of the nice things about spack is that it'll be easy and quicker to try out different TPL versions. |
Wow, that's nice. To be honest I don't know much about spack! I'm looking forward to learn how to use it! |
4ce2bc1
to
f00557b
Compare
47039b1
to
944e289
Compare
Closes #612 |
@bmhan12 @TotoGaz I changed how the host configs are laid out, how do I change the docker configs? https://travis-ci.com/github/GEOSX/GEOSX/jobs/434157258#L236 |
You can change the docker host-config and configuration options in |
I have this kind of errors:
Why would Also I don't get it: the TPLs are still built with our classical |
@TotoGaz pretty much every unit test links to If you look I made some changes to |
1c3e0c3
to
12dcd8a
Compare
d710e0a
to
a883eaf
Compare
@rrsettgast all the Tavis tests passed (aside from the submodule check) and the tpls built on Quartz and Lassen. |
# | ||
# Performance portability | ||
# | ||
if(EXISTS ${GEOSX_TPL_DIR}/raja) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and if GEOSX_TPL_DIR
does not exist?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is only meant to be included when you define GEOSX_TPL_DIR
. Once you get to setupGEOSXThirdParty.cmake
if RAJA_DIR
isn't defined it'll error out because it's a hard dependency.
set(ENABLE_GEOSX_PTP ON CACHE BOOL "" FORCE) | ||
|
||
set(GEOSX_TPL_DIR "/usr/local/GEOSX/GEOSX_TPL" CACHE PATH "" FORCE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will break the build on my laptop won't it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it will. Eventually with Spack we will have a separate host-config for each platform. We can probably do it in such a way (untar the TPLs inside the repo for example) that you could use the same host config.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I take that back, in theory you could make it so that you could use the travis OSX host-config, since you'll be building the TPLs with spack anyways you should just use the host-config that spack spits out. We should only put host-configs in the repo for situations where we don't expect users to build the TPLs (LC and Docker).
Hello Ben, Sorry for the late answer. To configure VTK was quite a mess. In our tpls; we don't configure it with MPI because we didn't use any MPI feature of VTK. @TotoGaz and I ran a lot of failed configuration in order to find the smallest number of component to include. There is a script in VTK that allow you to find the dependencies of a file you wrote, outputting the CMake command to configure VTK to be able to compile this file. Please have a look to |
9709cb3
to
fb8acf6
Compare
I had to remove the |
The goal of this PR is to get a spack build working on Quartz and Lassen. Subsequent PRs can get it working on OSX and standard Linux systems. Finally replacing
thirdPartyLibs
will require writing some automated scripts to build the appropriate configs.Packages
expat
,freetype
,glew
,hdf5
,jpeg
, ...). Either this is not the same VTK we're using or we'll need to create a stripped down version. Since this is an optional TPL I will probably not port it in this PR.Platforms
Related to
https://github.com/GEOSX/integratedTests/pull/104