Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy mesh information of local parameters or LS reference (multimat) #104

Merged
merged 6 commits into from
Feb 26, 2024

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
19 checks passed
@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