search

loganoz / horses3d

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

Fix/cf #192

Closed loganoz closed 8 months ago

loganoz commented 8 months ago

@oscarmarino , it seems like the export surfaces only works in serial right?

oscarmarino commented 8 months ago

it should work in parallel, I've used it in cesvima with hybrib OpenMP+MPI and worked. It was intel and METIS, but don't think it should change.

loganoz commented 8 months ago

Well, MPI with intel release and Metis does work also in the CI, so I guess it's a minor thing

loganoz commented 8 months ago

@oscarmarino , here more info about the problem:

2023-11-08T09:28:23.1956403Z Program received signal SIGSEGV: Segmentation fault - invalid memory reference. 2023-11-08T09:28:23.1956416Z 2023-11-08T09:28:23.1956507Z Backtrace for this error: 2023-11-08T09:28:23.1956597Z #0 0x7f7bcfe23970 in ??? 2023-11-08T09:28:23.1956690Z #1 0x7f7bcfe22ad5 in ??? 2023-11-08T09:28:23.1956776Z #2 0x7f7bcfa4251f in ??? 2023-11-08T09:28:23.1956896Z #3 0x557343c5a497 in surfaceprepareforio 2023-11-08T09:28:23.1957222Z at /home/runner/work/horses3d/horses3d/Solver/src/libs/mesh/SurfaceMesh.f90:926 2023-11-08T09:28:23.1957383Z #4 0x557343c78587 in surfacemesh_MOD_surfconstruct 2023-11-08T09:28:23.1957688Z at /home/runner/work/horses3d/horses3d/Solver/src/libs/mesh/SurfaceMesh.f90:307 2023-11-08T09:28:23.1957842Z #5 0x5573437319e2 in dgsemclass_MOD_constructdgsem 2023-11-08T09:28:23.1958186Z at /home/runner/work/horses3d/horses3d/Solver/src/libs/discretization/DGSEMClass.f90:299 2023-11-08T09:28:23.1958291Z #6 0x5573433c87b6 in horses3dmainns 2023-11-08T09:28:23.1958604Z at /home/runner/work/horses3d/horses3d/Solver/src/NavierStokesSolver/main.f90:101 2023-11-08T09:28:23.1958760Z #7 0x5573433c93b9 in main 2023-11-08T09:28:23.1959109Z at /home/runner/work/horses3d/horses3d/Solver/src/NavierStokesSolver/main.f90:3

I will merge the branch, as the problem is in master already, only that we are not testing for it.