search

loganoz / horses3d

HORSES3D: A high-order discontinuous Galerkin solver for flow simulations and multi-physics applications
https://loganoz.github.io/horses3d/
MIT License
112 stars 24 forks source link

Fix array bounds in `UpdateValues` #106

Closed Andres-MG closed 1 year ago

Andres-MG commented 1 year ago

The lower limits needed an offset of +1 to avoid overwriting part of the previous results. Additionally, I have optimized the subroutine by fusing the three loops into one and computing the divisions only once.

Maybe @oscarmarino could also check if this also fixes #96 .

oscarmarino commented 1 year ago

Seems that has been fixed the issue.