search

pecos / tps

Torch Plasma Simulator
BSD 3-Clause "New" or "Revised" License
8 stars 2 forks source link

GPU support for averaging #48

Closed marc-85 closed 2 years ago

marc-85 commented 3 years ago

With these modifications averaging happens now on GPU with no data movement CPU-GPU. The tests uses the same reference solutions as those used for the CPU tests. Tests performed with multiple MPI processes are skipped when not enough GPUs are found

trevilo commented 2 years ago

Closing because this is too stale. It is unclear to me exactly what change was intended here. The title and first comment imply that it contains an extention of the averaging on the gpu, but from the commits referenced I don't see it. But... the diff is also very weird, since somehow github thinks 0 files have been changed.

Regardless, we are doing averaging on the gpu (as of #67), so at least that part has been handled.

The only non-merge, non-style commit here appears to be 4bbb6c7, which fixes a variable name in the mean. Since this PR is too stale to be useful otherwise, I'm closing it and opening an issue about the name (#157).