-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Add heFFTe #26633
Add heFFTe #26633
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/heffte:
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/heffte:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
4fe5d74
to
11d0ed6
Compare
@conda-forge/help-c-cpp this new package for heFFTe is ready for review. In the Linux test, I do not understand the linker missing the
|
512dede
to
d34f566
Compare
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/heffte:
|
ping @conda-forge/help-c-cpp any idea on this pthread issue in tests? :) |
I agree to be listed as feedstock maintainer. |
Is this possibly an issue with FFTW's CMake support itself? Because that is not officially supported, see e.g. FFTW/fftw3#307, so you may need to fix it there. |
That is possible, I myself do some quite elaborate back-and-forth when searching for FFTW downstream in other codes: https://github.com/ECP-WarpX/WarpX/blob/development/cmake/dependencies/FFT.cmake @mkstoyanov do you have any recommendations on how to build heFFTe against FFTW on Linux or have seen this issue before? |
@ax3l I don't know why I missed the notification on this mention. Feel free to email me at work and I seem to be getting the comments on the heFFTe issues. I've only ever build heFFTe with FFTW on Linux using either default installed packages from The long list of "overlinking" was a work-around when I don't know what will be installed. The hope is that the compiler will just pick the first one it finds and ignores the rest. I don't know what is the right logic to handle this if we have multiple libraries (e.g., openmp and threads) and how to detect whether a compiler supports OpenMP (clang can be iffy with OpenMP). The last-resort solution that heFFTe supports is that the user can manually specify:
If those two are present, heFFTe will not do any "guessing" or try to find or enable anything else. I don't know how/if you can get the list of libraries and includes from conda but if you can, this should just work. |
Add the heFFTe library, a Highly Efficient FFT for Exascale.
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Thanks, @mkstoyanov! I now started a patch to make the FFTW threads search in heFFTe's CMake logic more robust:
|
f91595e
to
370b92a
Compare
One test, `example_r2c`, likes to run with 4 processes. We have 2 cores on Azure.
@egpbos @jakirkham we made it - heFFTe ready for review & merge :) |
Awesome @ax3l! I'm not a reviewer here, but it LGTM nonetheless ;) |
Oops, sorry 😅 @conda-forge/help-c-cpp heFFTe package is ready to gooo 🚀 |
Co-authored-by: Isuru Fernando <[email protected]>
Thank you, Isuru! |
Add the heFFTe library, a Highly Efficient FFT for Exascale.
Checklist
If static libraries are linked in, the license of the static library is packaged.url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).