From f303199ffa31af619dc2dc17399f72d7371c0108 Mon Sep 17 00:00:00 2001 From: laurence Date: Wed, 27 Mar 2013 17:14:57 +0000 Subject: [PATCH 1/2] BUG: PatchTools::sortedEdgeFaces(): vector now normalised before dot product --- .../PatchTools/PatchToolsSortEdges.C | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/OpenFOAM/meshes/primitiveMesh/PatchTools/PatchToolsSortEdges.C b/src/OpenFOAM/meshes/primitiveMesh/PatchTools/PatchToolsSortEdges.C index 54306d7230..0877a50b5f 100644 --- a/src/OpenFOAM/meshes/primitiveMesh/PatchTools/PatchToolsSortEdges.C +++ b/src/OpenFOAM/meshes/primitiveMesh/PatchTools/PatchToolsSortEdges.C @@ -77,10 +77,12 @@ Foam::PatchTools::sortedEdgeFaces { if (f0[fpI] != e.start()) { - const vector faceEdgeDir = localPoints[f0[fpI]] - edgePt; - const scalar angle = faceEdgeDir & e2; + vector faceEdgeDir = localPoints[f0[fpI]] - edgePt; + faceEdgeDir /= mag(faceEdgeDir) + VSMALL; - if (angle < maxAngle) + const scalar angle = e2 & faceEdgeDir; + + if (mag(angle) < maxAngle) { maxAngle = angle; maxAngleEdgeDir = faceEdgeDir; @@ -114,10 +116,12 @@ Foam::PatchTools::sortedEdgeFaces { if (f[fpI] != e.start()) { - const vector faceEdgeDir = localPoints[f[fpI]] - edgePt; - const scalar angle = faceEdgeDir & e2; + vector faceEdgeDir = localPoints[f[fpI]] - edgePt; + faceEdgeDir /= mag(faceEdgeDir) + VSMALL; - if (angle < maxAngle) + const scalar angle = e2 & faceEdgeDir; + + if (mag(angle) < maxAngle) { maxAngle = angle; maxAngleEdgeDir = faceEdgeDir; From 179085af9745c483aedb8dfb8ca4de132e9c6665 Mon Sep 17 00:00:00 2001 From: mattijs Date: Thu, 28 Mar 2013 10:26:25 +0000 Subject: [PATCH 2/2] ENH: renumberMesh: sort face/cellZones --- .../mesh/manipulation/renumberMesh/renumberMesh.C | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/applications/utilities/mesh/manipulation/renumberMesh/renumberMesh.C b/applications/utilities/mesh/manipulation/renumberMesh/renumberMesh.C index d03f24cd7c..af968f8120 100644 --- a/applications/utilities/mesh/manipulation/renumberMesh/renumberMesh.C +++ b/applications/utilities/mesh/manipulation/renumberMesh/renumberMesh.C @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2011-2012 OpenFOAM Foundation + \\ / A nd | Copyright (C) 2011-2013 OpenFOAM Foundation \\/ M anispulation | ------------------------------------------------------------------------------- License @@ -494,7 +494,13 @@ autoPtr reorderMesh newFlipMap[i] = fZone.flipMap()[i]; } } - fZone.resetAddressing(newAddressing, newFlipMap); + labelList newToOld; + sortedOrder(newAddressing, newToOld); + fZone.resetAddressing + ( + UIndirectList