search

awslabs / palace

3D finite element solver for computational electromagnetics
https://awslabs.github.io/palace/dev
Apache License 2.0
224 stars 50 forks source link

Fix timer value of `Timer::TOTAL` for AMR simulations #225

Closed sebastiangrimberg closed 2 months ago

sebastiangrimberg commented 2 months ago

Fix after https://github.com/awslabs/palace/pull/212. The total time value for AMR simulations, when reduction is performed more than once, was previously double counting and printing an incorrect value.

kllrak commented 2 months ago

Did not build to try this, but the logic of the changes makes sense to me and should fix what I was seeing.