BUG: singleProcessorFaceSets: fix parallel/. Fixes #2359.

The logic was not maintaining consistent sets of constraints
on different processors. A single processor with a full
match (very easy with 0 local faces) would invalidate
adding the constraint.

src/parallel/decompose/decompositionMethods/decompositionConstraints/singleProcessorFaceSets/singleProcessorFaceSetsConstraint.C

@@ -6,7 +6,7 @@
      \\/     M anipulation  |
 -------------------------------------------------------------------------------
     Copyright (C) 2015-2016 OpenFOAM Foundation
-    Copyright (C) 2018-2019 OpenCFD Ltd.
+    Copyright (C) 2018-2019,2022 OpenCFD Ltd.
 -------------------------------------------------------------------------------
 License
     This file is part of OpenFOAM.
@@ -143,7 +143,18 @@ void Foam::decompositionConstraints::singleProcessorFaceSets::add
         const labelList& faceLabels = specifiedProcessorFaces[setI];
         for (const label facei : faceLabels)
         {
-            faceToSet[facei] = setI;
+            if (faceToSet[facei] == -1)
+            {
+                faceToSet[facei] = setI;
+            }
+            else if (faceToSet[facei] != setI)
+            {
+                WarningInFunction << "Face " << facei
+                    << " at " << mesh.faceCentres()[facei]
+                    << " is already in existing constraint "
+                    << faceToSet[facei]
+                    << endl;
+            }
         }
     }
 
@@ -190,7 +201,7 @@ void Foam::decompositionConstraints::singleProcessorFaceSets::add
             }
         }
 
-        reduce(store, andOp<bool>());
+        reduce(store, orOp<bool>());
 
         if (store)
         {

[alex]

Hello, I run this feature on v2412 and I get this warning:

--> FOAM Warning : Using 'singleProcessorFaceSets' constraint specification. This constraint keyword is 126 months old.

Would it be any update to consider?

Regards, Oli

here is my decomposeParDict: ` FoamFile { version 2.0; format ascii; class dictionary; object decomposeParDict; } // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * // numberOfSubdomains 12;

method hierarchical; coeffs { n (4 1 3);}

singleProcessorFaceSets ((AMIfaceSet -1)); constraints { } `

[Prashant Sonakar]

Hi @Oli772 ,

You may use constraints.

However, please note that we don't recommend using this platform for support related issues - refer https://develop.openfoam.com/Development/openfoam/-/wikis/Submitting-issues

/Prashant