View Issue Details

IDProjectCategoryView StatusLast Update
0001818OpenFOAM[All Projects] Bugpublic2015-08-08 14:32
ReporterssssAssigned Tohenry 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformGNU/LinuxOSUbuntuOS Version14.04
Product Versiondev 
Fixed in Version 
Summary0001818: New blockMesh patch face matching to be vertex ordering independent functionality appears broken
DescriptionThe new blockMesh patch face matching to be vertex ordering independent functionality doesn't work completely as expected.

In the tutorial icoFoam/cavity if one changes the order of the boundary face vertices of the movingWall the movingWall patch is not recognized and the movingWall faces are added to the the defaultFaces patch.

The original definition for the movingWall patch is:

movingWall
{
   type wall;
   faces
   (
      (3 7 6 2)
   );
}

With this vertices order the blockMesh utility throws these patches:

----------------
Patches
----------------
  patch 0 (start: 760 size: 20) name: movingWall
  patch 1 (start: 780 size: 60) name: fixedWalls
  patch 2 (start: 840 size: 800) name: frontAndBack

End

Showing that the movingWall faces are correctly identified. Instead if this vertices' order is used:

movingWall
{
   type wall;
   faces
   (
      (7 3 6 2)
   );
}

Then the blockMesh utility throws these patches:

----------------
Patches
----------------
  patch 0 (start: 760 size: 0) name: movingWall
  patch 1 (start: 760 size: 60) name: fixedWalls
  patch 2 (start: 820 size: 800) name: frontAndBack
  patch 3 (start: 1620 size: 20) name: defaultFaces

End

Where you can see that the movingWall faces are not recognized.

Instead if the order of the vertices is permuted as:

movingWall
{
   type wall;
   faces
   (
      (7 3 2 6)
   );
}

Then the blockMesh utility throws these patches:

----------------
Patches
----------------
  patch 0 (start: 760 size: 20) name: movingWall
  patch 1 (start: 780 size: 60) name: fixedWalls
  patch 2 (start: 840 size: 800) name: frontAndBack

End

Meaning that it works as intedend.

Steps To ReproduceModify the blockMesh file of the tutorial icoFoam/cavity changing the face vertices ordering on the movingWall patch. One can observe that the new blockMesh patch face matching to be vertex ordering independent functionality doesn't work as expected.

Additional InformationUsing the fastMerge option doesn't make any difference
TagsNo tags attached.

Activities

henry

2015-08-08 14:32

manager   ~0005210

Thanks for the bug-report.
Resolved by commit 23f4ce98718d3b7f32dceab8f3c8f7571eea36cf

Issue History

Date Modified Username Field Change
2015-08-08 12:38 ssss New Issue
2015-08-08 14:32 henry Note Added: 0005210
2015-08-08 14:32 henry Status new => resolved
2015-08-08 14:32 henry Resolution open => fixed
2015-08-08 14:32 henry Assigned To => henry