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

add script that creates singularity image for UWLCM with GPU support #77

Merged
merged 4 commits into from
Jul 18, 2019

Conversation

trontrytel
Copy link
Contributor

Thanks to great help from @mwarusz I'm finally done with the singularity image for UWLCM.

It is based on ubuntu 18.04 and cuda 10.0. Hope this helps some future users.

In the pull I just add the script that creates it and instructions how to do it. Let me know if you want the image too. Or if you want to try to create it yourself and test things. I'm assuming that both the libraries and UWLCM may still change. Therefore they are not part of the image but have to be installed locally later inside the image. If thats not the case they could be added to it when the image is build.

A side note 1: I also tried with an image based on ubuntu 16.04 and cuda 8.0 but I got some errors when compiling the moist thermal in UWLCM. I'm leaving it for now since it works with the newer versions. But in case you want to run it on a machine with older cuda version its probably not far from working too. Let me know if you want that script too.

A side note 2: It takes extremely long time to compile UWLCM. Maybe it would be useful to suggest a minimal/default set of options to compile. And ignore the others.

@trontrytel trontrytel requested a review from pdziekan July 16, 2019 18:18
@trontrytel
Copy link
Contributor Author

Also thank are due to @djarecka who helped me get started with Singularity :) @djarecka - here is the final solution...

@pdziekan
Copy link
Contributor

I've tested building and using the image and it works! Thanks for the contribution.

I have two small suggestions:

  1. Move singularity files (script and readme) to a separate folder.

  2. In singularity readme specify install location with
    cmake -DCMAKE_INSTALL_PREFIX=/local_folder/
    instead of
    make install DESTDIR=/local_folder/.
    Using INSTALL_PREFIX has the advantage that it gives correct rpath in the python bindings of libcloudph++

@trontrytel
Copy link
Contributor Author

Done. @pdziekan , since we are looking at the installation stuff. Would it make sense to put both libmpdata and libcloud config files in the same place?

Right now:
Dlibmpdata++_DIR=/local_folder/usr/local/share/libmpdata++ but
Dlibcloudph++_DIR=/local_folder/usr/local/lib/cmake/libcloudph++.

The symmetry and the overall order in the world would improve.

@trontrytel
Copy link
Contributor Author

I've tested building and using the image and it works!

I must say I almost don't believe that it worked :)

@pdziekan
Copy link
Contributor

libcloud PR changing install location of cmake config to share/:
igfuw/libcloudphxx#377

@pdziekan pdziekan merged commit 93591eb into igfuw:master Jul 18, 2019
@trontrytel trontrytel deleted the singularity branch July 23, 2019 20:17
ryba183 pushed a commit to ryba183/UWLCM that referenced this pull request Dec 9, 2020
add script that creates singularity image for UWLCM with GPU support
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.

None yet

2 participants