WarpX icon indicating copy to clipboard operation
WarpX copied to clipboard

Published 20 hours ago • verified publisher icon ECP-WarpX

remove unnecessary limit check in UpdateMomentumPerezElastic

Open JustinRayAngus opened this issue 1 year ago • 2 comments
trafficstars

Third PR of several related to cleaning/streamlining UpdateMomentumPerezElastic.

JustinRayAngus avatar Aug 28 '24 20:08 JustinRayAngus

Context: we introduced this to prevent NaNs in #4304

It looks like this could still apply. If it does, let us add an inline comment on the if condition.

ax3l avatar Aug 28 '24 21:08 ax3l

Thanks for the comment @ax3l. There is no division in these lines by vcms. So it is safe to remove the limit check.

JustinRayAngus avatar Aug 28 '24 21:08 JustinRayAngus