pecos / tps

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

Call FaceElementTransformations::SetIntPoint in boundary area calc #169

Closed trevilo closed 2 years ago

trevilo commented 2 years ago

This PR fixes a long standing issue in the boundary area calculation. Before using the FaceElementTransformations object, we have to set the integration point by calling SetIntPoint, which we had failed to do. Since we're only using the FaceElementTransformations::Weight function, which won't actually depend on the quadrature point for linear elements, it seems we were getting away with it... until #168.

After b5ea27a, I have been unable to reproduce #168, so this PR resolves #168.