Merge pull request #625 from adtzlr/fix-mesh-flip

Fix `Mesh.flip(mask=None)`: Take care of the mask
adtzlr · Feb 16, 2024 · cd15db1 · cd15db1
2 parents cafae00 + cac217b
commit cd15db1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -16,6 +16,9 @@ All notable changes to this project will be documented in this file. The format
 ### Changed
 - Don't disconnect the dual mesh by default for regions `RegionQuadraticTriangle` and `RegionQuadraticTetra` in `FieldsMixed`.
 
+### Fixed
+- Fix `Mesh.flip(mask=None)`: Take care of the mask (it wasn't applied to the cells-array of the mesh).
+
 ## [7.17.0] - 2024-02-15
 
 ### Added

diff --git a/src/felupe/mesh/_tools.py b/src/felupe/mesh/_tools.py
@@ -705,7 +705,7 @@ def flip(points, cells, cell_type, mask=None):
     cells_new = cells.copy()
 
     for face in faces_to_flip:
-        cells_new[:, face] = cells[:, face[::-1]]
+        cells_new[mask, face] = cells[mask, face[::-1]]
 
     return points, cells_new, cell_type