search

conda-forge / plumed-feedstock

A conda-smithy repository for plumed.
BSD 3-Clause "New" or "Revised" License
0 stars 9 forks source link

MPI enabled builds #20

Open awvwgk opened 3 years ago

awvwgk commented 3 years ago

Issue: Currently looking into building DFTB+ on conda-forge with plumed support, I think we would need an MPI enabled version as well to build all variants of the package with plumed support. Are there plans to support MPI in this package as well?

cc @conda-forge/dftbplus

GiovanniBussi commented 3 years ago

@awvwgk thanks for raising this. Currently we have no plan in this sense. I noticed that unfortunately preparing a package that works both with and without MPI is non trivial, and I thus didn't have time to figure out how to fix our recipe. If you know how to do it, could you please make a pull request in this sense? Ideally:

Thanks!

awvwgk commented 3 years ago

Will give it a try in #21, let's see how this goes.

There is probably more work to do than adding MPI to the build matrix. Do any of the dependencies have to support MPI as well?

GiovanniBussi commented 3 years ago

There is probably more work to do than adding MPI to the build matrix. Do any of the dependencies have to support MPI as well?

Not that I know. I guess you are thinking about fftw? We only use the serial version.

GiovanniBussi commented 3 years ago

Will give it a try in #21, let's see how this goes.

🤞

awvwgk commented 3 years ago

Seems like it works with MPICH, but fails with OpenMPI. Not sure what the cause it, but maybe it seems to be an issue with the compiler wrappers (we don't use those in DFTB+).

awvwgk commented 3 years ago

Feel free to close this PR or keep it open to track the build failure on OSX-ARM with OpenMPI.