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

[WIP] Experiment in streamlining travis #1698

Closed
wants to merge 25 commits into from

Conversation

jchodera
Copy link
Member

This is the beginning of an experiment to streamline travis and produce linux, source, and osx build artifacts using docker images.

Currently, no testing or S3 docs deployment is done.

@peastman
Copy link
Member

What are you doing working on this? It's Christmas!

@jchodera
Copy link
Member Author

Hopefully saving myself from having to spend an actual vacation building more release artifacts. :)

@jchodera
Copy link
Member Author

Tests run inside the docker container don't fail gracefully because CUDA is installed but no CUDA devices are available:

>       this = _openmm.new_Context(*args)
E       Exception: St8bad_cast
/miniconda/lib/python3.5/site-packages/simtk/openmm/openmm.py:4035: Exception
----------------------------- Captured stderr call -----------------------------
NVIDIA: no NVIDIA devices found
NVIDIA: no NVIDIA devices found
NVIDIA: no NVIDIA devices found
______________________ TestModeller.test_addHydrogenspH11 ______________________

Is there a way around those failures? Or is it best to build a conda package in the docker container, install it in the travis environment (outside the container) directly, and test that?

@peastman
Copy link
Member

You can turn off OPENMM_BUILD_CUDA_TESTS.

@jchodera jchodera mentioned this pull request Jan 17, 2017
@jchodera
Copy link
Member Author

jchodera commented Dec 5, 2017

This might be worth revisiting at some point since it shaved 40 minutes off the total build time (from 1h28m to 50m total time).

@jchodera jchodera closed this Apr 12, 2019
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