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

Add support for all heFFTe MPI communication backends #799

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

rodrigoacb
Copy link

@rodrigoacb rodrigoacb commented Oct 30, 2024

This PR extends the available backends available in heFFTe using a enum class to pick the chosen reshape/communication pattern. 

Previously supported: alltoallv and p2p

This adds alltoall and p2p_plined

This changes were done in cooperation with @rhalver

Rodrigo Bartolomeu and others added 2 commits October 30, 2024 16:56
@streeve
Copy link
Member

streeve commented Oct 31, 2024

Thanks for the improvements! Do you mind splitting into two PRs? I think I'll want someone else to at least co-review the spline details, but the FFT should be quick to merge

@rodrigoacb
Copy link
Author

Thanks for the improvements! Do you mind splitting into two PRs? I think I'll want someone else to at least co-review the spline details, but the FFT should be quick to merge

Hi @streeve Sam! We also thank you for maintaining Cabana! I don't mind splitting the PRs. Should I leave this one for the FFT and create a new one for the Splines?

I had a look into the actions, and I think I have already addressed it with fixes that I can commit once the PRs are at the places.

@streeve
Copy link
Member

streeve commented Nov 4, 2024

Thanks for the improvements! Do you mind splitting into two PRs? I think I'll want someone else to at least co-review the spline details, but the FFT should be quick to merge

Hi @streeve Sam! We also thank you for maintaining Cabana! I don't mind splitting the PRs. Should I leave this one for the FFT and create a new one for the Splines?

I had a look into the actions, and I think I have already addressed it with fixes that I can commit once the PRs are at the places.

Yes, that sounds great!

@rodrigoacb rodrigoacb changed the title Add Splines of order 4 to 7 and support for all heFFTe MPI communication backends Add support for all heFFTe MPI communication backends Nov 5, 2024
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.

3 participants