-
Notifications
You must be signed in to change notification settings - Fork 4
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
caching openmpi, .julia, rm Simulation #21
Conversation
SCRC-87 See failure https://travis-ci.org/github/ScottishCovidResponse/Simulation.jl/jobs/683003127 Julia tries to "Activating new environment at `~/build/ScottishCovidResponse/Simulation.jl/openmpi/Project.toml`" - openmpi is in the wrong place for caching
SCRC-87 Should have gone in with previous commit to this branch
SCRC-87 ERROR: LoadError: LoadError: InitError: could not load library "/home/travis/build/ScottishCovidResponse/Simulation.jl/openmpi/lib/libmpi" which isn't where openmpi should be - should be in ~
SCRC-87 print statements at top of test/runtests.jl that need to be removed eventually print statements in .travis.yml ""
SCRC-87 caching openmpi dir, and moving it into the JULIA_PROJECT path at the start of the travis script
SCRC-87 caching openmpi dir, and moving openmpi dir to cache
SCRC-87 ERROR: LoadError: LoadError: InitError: MPI library has changed, please restart Julia. was copying mpi stuff into Simulation.jl, now setting JULIA_MPI_PATH
SCRC-87 wasnt checking in right palce for mpirun
SCRC-87 was only deleting the .julia/packages/Simulation.jl after_success and not after_success. now done in before_cache:. Can't do it in after_script because that happens after cache:.
SCRC-87 had ./openmpi not ~/openmpi
SCRC-87 good old print statements
SCRC-87 working on installMPI.sh for macosx, trying to get it to recognise the cached directory /usr/local/Cellar
SCRC-87 working on installMPI.sh for macosx, trying to get it to recognise the cached directory /usr/local/Cellar
SCRC-87 working on installMPI.sh for macosx, got wrong path to mpi installation under Cellar
SCRC-87 need to set PATH and LD_LIBRARY_PATH
SCRC-87 set DYLD_LIBRARY_PATH not LD_LIBRARY_PATH
SCRC-87 might need to manually set DYLD_LIBRARY_PATH now attempting to relink with brew
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.
Hi James, I have lost track a bit, is this ready to be merged?
SCRC-87 trying different install path so maybe fewer symlinks to deal with?
Codecov Report
@@ Coverage Diff @@
## dev #21 +/- ##
==========================================
+ Coverage 26.89% 29.29% +2.39%
==========================================
Files 46 46
Lines 2506 2550 +44
==========================================
+ Hits 674 747 +73
+ Misses 1832 1803 -29
Continue to review full report at Codecov.
|
Not ready yet I'm afraid :( |
SCRC-87 travis says /Users/travis/openmpi is brought out of cache, yet ls reveals nothing
No worries! Sorry I was just going through open PRs last night, but now with my brain switched on I see you need to run travis twice on these commits |
SCRC-87 it was in .travis.yml but makes more sense in installMPI.sh
…12-04-08-30-704-3078825718 CompatHelper: bump compat for "Plots" to "1.11"
This might well speed up travis, but will have to run travis twice to find out.