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

Use command line compiler if it is explicitly specified #3018

Merged
merged 2 commits into from
Feb 11, 2021

Conversation

peastman
Copy link
Member

@peastman peastman commented Feb 9, 2021

In 7.5 we changed the behavior of compiling CUDA kernels, so it would prefer the runtime compiler over the command line compiler. This was supposed to avoid problems, but it ended up creating some new problems. One of them is that if anything went wrong with the runtime compiler, it was hard to fix. There was no straightforward way to force it to use the command line compiler instead.

This PR changes the behavior so that if you explicitly specify a command line compiler, either with the 'CudaCompiler' property or the OPENMM_CUDA_COMPILER environment variable, that forces it to use the compiler you specify. It will never use the runtime compiler instead.

@peastman
Copy link
Member Author

run jenkins

@peastman
Copy link
Member Author

@swails Jenkins is failing to start up:

+ docker inspect -f . swails/openmm-all:latest
.
Failed to run image 'swails/openmm-all:latest'. Error: docker: Error response from daemon: could not select device driver "" with capabilities: [[gpu]].

@swails
Copy link
Contributor

swails commented Feb 11, 2021

@swails Jenkins is failing to start up:

I was bringing another agent into the Jenkins array that needed to be rebooted. The PR job has now passed.

@peastman
Copy link
Member Author

Thanks!

@peastman peastman merged commit 6d4abc5 into openmm:master Feb 11, 2021
@peastman peastman deleted the compiler branch February 11, 2021 16:19
@peastman peastman mentioned this pull request Feb 11, 2021
7 tasks
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