BUG: polyMesh: do count coupled faces for non-closedness

This commit is contained in:
mattijs
2013-02-06 09:43:30 +00:00
parent 249ddd6a09
commit 6ba4f9ceb7
3 changed files with 15 additions and 16 deletions

View File

@ -2,7 +2,7 @@
========= | ========= |
\\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / F ield | OpenFOAM: The Open Source CFD Toolbox
\\ / O peration | \\ / O peration |
\\ / A nd | Copyright (C) 2011-2012 OpenFOAM Foundation \\ / A nd | Copyright (C) 2011-2013 OpenFOAM Foundation
\\/ M anipulation | \\/ M anipulation |
------------------------------------------------------------------------------- -------------------------------------------------------------------------------
License License
@ -576,9 +576,6 @@ public:
// Geometric checks. Selectively override primitiveMesh functionality. // Geometric checks. Selectively override primitiveMesh functionality.
//- Check boundary for closedness
virtual bool checkClosedBoundary(const bool report = false) const;
//- Check non-orthogonality //- Check non-orthogonality
virtual bool checkFaceOrthogonality virtual bool checkFaceOrthogonality
( (

View File

@ -2,7 +2,7 @@
========= | ========= |
\\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / F ield | OpenFOAM: The Open Source CFD Toolbox
\\ / O peration | \\ / O peration |
\\ / A nd | Copyright (C) 2012 OpenFOAM Foundation \\ / A nd | Copyright (C) 2012-2013 OpenFOAM Foundation
\\/ M anipulation | \\/ M anipulation |
------------------------------------------------------------------------------- -------------------------------------------------------------------------------
License License
@ -516,15 +516,17 @@ bool Foam::polyMesh::checkCellDeterminant
} }
bool Foam::polyMesh::checkClosedBoundary(const bool report) const //- Could override checkClosedBoundary to not look at (collocated!) coupled
{ // faces
return primitiveMesh::checkClosedBoundary //bool Foam::polyMesh::checkClosedBoundary(const bool report) const
( //{
faceAreas(), // return primitiveMesh::checkClosedBoundary
report, // (
syncTools::getInternalOrCoupledFaces(*this) // faceAreas(),
); // report,
} // syncTools::getInternalOrCollocatedCoupledFaces(*this)
// );
//}
bool Foam::polyMesh::checkFaceOrthogonality bool Foam::polyMesh::checkFaceOrthogonality

View File

@ -2,7 +2,7 @@
========= | ========= |
\\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / F ield | OpenFOAM: The Open Source CFD Toolbox
\\ / O peration | \\ / O peration |
\\ / A nd | Copyright (C) 2012 OpenFOAM Foundation \\ / A nd | Copyright (C) 2012-2013 OpenFOAM Foundation
\\/ M anipulation | \\/ M anipulation |
------------------------------------------------------------------------------- -------------------------------------------------------------------------------
License License
@ -64,7 +64,7 @@ bool Foam::primitiveMesh::checkClosedBoundary
for (label faceI = nInternalFaces(); faceI < areas.size(); faceI++) for (label faceI = nInternalFaces(); faceI < areas.size(); faceI++)
{ {
if (internalOrCoupledFaces.size() && !internalOrCoupledFaces[faceI]) if (!internalOrCoupledFaces.size() || !internalOrCoupledFaces[faceI])
{ {
sumClosed += areas[faceI]; sumClosed += areas[faceI];
sumMagClosedBoundary += mag(areas[faceI]); sumMagClosedBoundary += mag(areas[faceI]);