Skip to content

Ouranos integration - #41

Closed
aureliendaut wants to merge 77 commits into
arcaneframework:dev/numpex-exadifrom
aureliendaut:dev/numpex-exadi
Closed

aureliendaut wants to merge 77 commits into
arcaneframework:dev/numpex-exadifrom
aureliendaut:dev/numpex-exadi

Conversation

@aureliendaut

@aureliendaut aureliendaut commented May 28, 2026 •

Copy link
Copy Markdown
Collaborator

What's done

  • Read a .meshb file via Ouranos and populate an Arcane IPrimaryMesh with tetrahedra and node coordinates
  • Generalized the sequential path (use_internal_partition=true) to handle multiple element types depending on mesh dimension: tetrahedra + hexahedra in 3D, triangles in 2D
  • Added the parallel reading path (use_internal_partition=false): each rank now reads and partitions the mesh itself via Ouranos (Mesh::to_part<Metis>)
  • Fixed several duplicate-cell / crash bugs in the parallel path (cell ownership, ghost layer depth, ghost node coordinates) — each element is now assigned to exactly one rank, correct for any number of ranks
  • Extended the parallel path to hexahedra (was tetrahedra-only)
  • Fixed centerCompute() in ExaDiBenchModule.cc (was dividing by the local cell count instead of the global one)
  • Added a per-rank load-balance log (min/max/total owned cells)
  • Registered ExaDiBench-meshb-hexa and ExaDiBench-meshb-hexa-ouranos as ctest targets — all 4 combinations (tetra/hexa x with/without Ouranos) now covered and passing
  • Replaced the structured hexahedral test mesh with a real unstructured one

Limitations

  • Parallel path still doesn't support 2D (triangles)
  • ctest only checks that runs don't crash, not that computed values are correct — no reference-based validation wired in yet

gratienj and others added 30 commits January 26, 2026 09:22
…e10-tests-with-alien-solvers

update ArcGeoSim
- fix convergence criteria in hypre ILU0 test
- add MCGSolver ILU0 and CprAMG test
- add PETSC ILU0 test
- clean un-necessary commented solver settings
- add smaller test cases for 1 core or 1 gpu tests
- add spe10 mcgsolver tests
- use relative error instead of absolute error
- mcgsolver gpu test activated if mcgsolver::gpu target present
…-name

target name is mcgsolver::cuda not mcgsolver::gpu
…-relative-validator

Use relative error validators
…aneframework#37)

* Remove Ubuntu 22.04 and 2018b workflows, and update 2021b images

* fix root user in 2021b workflows

* - use relative error validator fors tests that use validators

* Remove Ubuntu 22.04 and 2018b workflows, and update 2021b images

* fix root user in 2021b workflows

* Update use-case-two-phase-flow.arc

- use relative error for validator

* - fix validator typo

* Workflows foss/2021b : allow run OpenMPI as root

---------

Co-authored-by: gazoppce <cedric.gazoppi@ifpen.fr>
Co-authored-by: guignont <thomas.guignon@ifpen.fr>
…pex-exadi

# Conflicts:
#	CMakeLists.txt
#	src/ExaDiBench/MeshbReader/MeshbMeshReaderService.cc
#	test/Authors.txt
#	test/CMakeLists.txt
#	test/ExaDiBench/README.md
#	test/ExaDiBench/bench-meshb.arc
#	test/TwoPhaseFlowSimulation/use-case-two-phase-flow.arc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants