Arturo Vargas

Results 130 comments of Arturo Vargas

Please delete branch if no longer needed.

Which reducer unit test are disabled for gpu policies?

I think this is great, but I do wonder if maybe the development should happen within the new reducer interface to start motivating folks to transition over.

Hi folks, just a friendly reminder about this PR.

Hi all, thanks for the review. Just got back from vacation, will start addressing comments/feedback this week.

> I noticed that `lor-transfer-p.cpp` uses "CPU time" (sum of all ranks times), which I think is what you report in the initial pull request? > > I suspect that...

> @artv3 I merged #4555 here. Hopefully everything still works properly :) can you confirm? And if you have some time to take care of the permuted indices that I...

> Uh oh. Is it a device-specific issue? I can't reproduce it locally (CPU only; seems to work for me) Yes, I think it is. On the cpu I get:...

> @v-dobrev @artv3, see [#4408 (comment)](https://github.com/mfem/mfem/pull/4408#discussion_r1821860778), this will crash on CUDA, it should be fixed before merging. Thanks for catching that, just pushed in a fix.