MPI.jl icon indicating copy to clipboard operation
MPI.jl copied to clipboard

add Ibcast!

Open PetrKryslUCSD opened this issue 1 year ago • 7 comments

A convenience function is added, Ibcast!.

PetrKryslUCSD avatar Sep 29 '24 18:09 PetrKryslUCSD

Thanks could you also add a test?

vchuravy avatar Sep 30 '24 08:09 vchuravy

And documentation.

giordano avatar Sep 30 '24 09:09 giordano

Ibcast! and Iallreduce! tests have been added. They also have a doc string. Is anything else required?

PetrKryslUCSD avatar Nov 17 '24 18:11 PetrKryslUCSD

Need to add the docstrings to the documentation

giordano avatar Nov 17 '24 19:11 giordano

Done.

PetrKryslUCSD avatar Nov 17 '24 21:11 PetrKryslUCSD

I just noticed this PR: https://github.com/JuliaParallel/MPI.jl/pull/827 It also adds Iallreduce!.

PetrKryslUCSD avatar Nov 18 '24 00:11 PetrKryslUCSD

@Keluaa : It'd be good to merge these PRs somehow...

PetrKryslUCSD avatar Nov 18 '24 00:11 PetrKryslUCSD

This PR would be really useful. It looks like the docs and tests were added. The logs have expired so can't now tell why the CI failed.

jwscook avatar Dec 22 '25 10:12 jwscook

The new test_iallreduce tests are segfaulting on both AMD and Nvidia GPUs

giordano avatar Dec 22 '25 11:12 giordano

Discussion of failing tests in https://github.com/JuliaParallel/MPI.jl/pull/827

PetrKryslUCSD avatar Dec 22 '25 17:12 PetrKryslUCSD