-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Fix singularity image #4376
Fix singularity image #4376
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4376 +/- ##
=======================================
Coverage 82.53% 82.53%
=======================================
Files 309 309
Lines 16017 16017
=======================================
Hits 13220 13220
Misses 2797 2797 Continue to review full report at Codecov.
|
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.
LGTM. We could upgrade to a reasonable PyTorch version in a later PR.
Added explanation comment for setting timezone to recipe. Should be good to go. |
Thank you! |
Purpose
Pull request fixes tzdata data-entry issue brought up in issue #4365.
Changelog
nvidia/cuda:10.1-cudnn7-devel-ubuntu18.04
.g++
was too old to compilenumpy
in the old base image.make
inapt-get install -y --no-install-recommends
.make
is provided by thebuild-essential
package.apt-get install -y --no-install-recommends
. Neededgfortran
,liblapack-dev
,libopenblas-dev
, andlibhdf5-dev
.Testing
Had some of the consultants I work with run tests on our cluster. Did not encounter any runtime errors.
What's next
Updating
Dockerfile
andsingularity
recipe to newer version of pytorch_geometric.