Skip to content

Conversation

@laetitia-m
Copy link
Contributor

  1. Replace memcopy by PMMG_copy_mmgInfo function to copy mesh information when merge the mesh in PMMG_mergeParmesh_rcvParMeshes. memcopy was copying the pointer address only and was leading to double free error.
  2. Modify PMMG_copy_mmgInfo to now update correctly the copy of mesh->info.mat, mesh->info.invmat.lookup and mesh->info.par arrays when we have a *mmg3d input file (local parameters or LS multimat references).
  3. Modify PMMG_mpisizeof_filenames, PMMG_mpipack_infos and PMMG_mpiunpack_infos to take into account mesh->info.invmat data correctly as the structure MMG5_InvMat was added to mmg after the initial creation of this function in ParMmg.
  4. Add ci tests to ensure the input file *mmg3d is read correctly for centralized/distributed input and centralized/distributed output + update gitlab tag to point towards the latest up-to-date testparmmg commit.

…e the mesh. memcopy was copying the pointer address only and was leading to double free error
…d par arrays when we have a *mmg3d input file for local parameters or LS references (multimat)
…k_infos to take into account mesh->info.invmat data correctly
… PMMG_mpiunpack_infos function with ier_mesh != 1
@Algiane Algiane added kind: bug Something isn't working kind: enhancement enhancement to an existing feature part: parmmg specific to parmmg code part part: MPI / HPC MPI or HPC related labels Feb 26, 2024
Copy link
Member

@Algiane Algiane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is all good for me,
Thanks!

@Algiane Algiane merged commit 11fec66 into MmgTools:develop Feb 26, 2024
@laetitia-m laetitia-m deleted the feature/mmg3d-input-file branch March 4, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: bug Something isn't working kind: enhancement enhancement to an existing feature part: MPI / HPC MPI or HPC related part: parmmg specific to parmmg code part

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants