From b1e7c009529072c5bb0f3fc2834e3ea0ef5cf27f Mon Sep 17 00:00:00 2001
From: andy <andy>
Date: Mon, 20 Jan 2014 13:15:10 +0000
Subject: [PATCH] ENH: AMI - improved output messages

---
 .../AMIInterpolation/AMIInterpolation.C       | 26 +++++++++++++------
 .../AMIMethod/AMIMethod/AMIMethod.C           | 11 ++------
 2 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/src/meshTools/AMIInterpolation/AMIInterpolation/AMIInterpolation.C b/src/meshTools/AMIInterpolation/AMIInterpolation/AMIInterpolation.C
index 16714be445..3dd2863e98 100644
--- a/src/meshTools/AMIInterpolation/AMIInterpolation/AMIInterpolation.C
+++ b/src/meshTools/AMIInterpolation/AMIInterpolation/AMIInterpolation.C
@@ -247,14 +247,17 @@ void Foam::AMIInterpolation<SourcePatch, TargetPatch>::normaliseWeights
                 << gMin(wghtSum) << ", "
                 << gMax(wghtSum) << ", "
                 << gAverage(wghtSum) << endl;
-        }
-    }
 
-    if (debug && nLowWeight)
-    {
-        Pout<< "AMI: Identified " << nLowWeight
-            << " faces with weights less than " << lowWeightTol
-            << endl;
+            const label nLow = returnReduce(nLowWeight, sumOp<label>());
+
+            if (nLow)
+            {
+                IInfo<< "AMI: Patch " << patchName
+                    << " identified " << nLow
+                    << " faces with weights less than " << lowWeightTol
+                    << endl;
+            }
+        }
     }
 }
 
@@ -784,6 +787,14 @@ void Foam::AMIInterpolation<SourcePatch, TargetPatch>::update
     const TargetPatch& tgtPatch
 )
 {
+    label srcTotalSize = returnReduce(srcPatch.size(), sumOp<label>());
+    label tgtTotalSize = returnReduce(tgtPatch.size(), sumOp<label>());
+
+    IInfo<< "AMI: Creating addressing and weights between "
+        << srcTotalSize << " source faces and "
+        << tgtTotalSize << " target faces"
+        << endl;
+
     // Calculate face areas
     srcMagSf_.setSize(srcPatch.size());
     forAll(srcMagSf_, faceI)
@@ -956,7 +967,6 @@ void Foam::AMIInterpolation<SourcePatch, TargetPatch>::update
     }
     else
     {
-
         // calculate AMI interpolation
         autoPtr<AMIMethod<SourcePatch, TargetPatch> > AMIPtr
         (
diff --git a/src/meshTools/AMIInterpolation/AMIInterpolation/AMIMethod/AMIMethod/AMIMethod.C b/src/meshTools/AMIInterpolation/AMIInterpolation/AMIMethod/AMIMethod/AMIMethod.C
index 4805878be2..eb9406c562 100644
--- a/src/meshTools/AMIInterpolation/AMIInterpolation/AMIMethod/AMIMethod/AMIMethod.C
+++ b/src/meshTools/AMIInterpolation/AMIInterpolation/AMIMethod/AMIMethod/AMIMethod.C
@@ -2,7 +2,7 @@
   =========                 |
   \\      /  F ield         | OpenFOAM: The Open Source CFD Toolbox
    \\    /   O peration     |
-    \\  /    A nd           | Copyright (C) 2013 OpenFOAM Foundation
+    \\  /    A nd           | Copyright (C) 2013-2014 OpenFOAM Foundation
      \\/     M anipulation  |
 -------------------------------------------------------------------------------
 License
@@ -337,14 +337,7 @@ Foam::AMIMethod<SourcePatch, TargetPatch>::AMIMethod
     tgtMagSf_(tgtMagSf),
     srcNonOverlap_(),
     triMode_(triMode)
-{
-    label srcSize = returnReduce(srcPatch_.size(), sumOp<label>());
-    label tgtSize = returnReduce(tgtPatch_.size(), sumOp<label>());
-
-    IInfo<< "AMI: Creating addressing and weights between "
-        << srcSize << " source faces and " << tgtSize << " target faces"
-        << endl;
-}
+{}
 
 
 // * * * * * * * * * * * * * * * * Destructor * * * * * * * * * * * * * * * //