Time: Removed graphFormat

setFormat no longer defaults to the value of graphFormat optionally set in
controlDict and must be set in the functionObject dictionary.

boundaryFoam, financialFoam and pdfPlot still require a graphFormat entry in
controlDict but this is now read directly rather than by Time.

applications/legacy/basic/financialFoam/financialFoam.C

@@ -82,7 +82,7 @@ int main(int argc, char *argv[])
 
         if (runTime.writeTime())
         {
-            setWriter::New(runTime.graphFormat())->write
+            setWriter::New(runTime.controlDict().lookup("graphFormat"))->write
             (
                 runTime.globalPath()
                /functionObjects::writeFile::outputPrefix

applications/modules/multiphaseEuler/functionObjects/populationBalanceSizeDistribution/populationBalanceSizeDistribution.C

@@ -408,11 +408,7 @@ bool Foam::functionObjects::populationBalanceSizeDistribution::read
 
     fvMeshFunctionObject::read(dict);
 
-    formatterPtr_ = setWriter::New
+    formatterPtr_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     return false;
 }

applications/solvers/boundaryFoam/makeGraphs.H

@@ -13,7 +13,7 @@ const volSymmTensorField R
 
 runTime.write();
 
-setWriter::New(runTime.graphFormat())->write
+setWriter::New(runTime.controlDict().lookup("graphFormat"))->write
 (
     runTime.globalPath()
    /functionObjects::writeFile::outputPrefix

applications/utilities/postProcessing/lagrangian/particleTracks/createFields.H

@@ -6,7 +6,4 @@ label sampleFrequency(propsDict.lookup<label>("sampleFrequency"));
 
 label maxPositions(propsDict.lookup<label>("maxPositions"));
 
-word setFormat
+word setFormat(propsDict.lookup<word>("setFormat"));
-(
-    propsDict.lookupOrDefault<word>("setFormat", runTime.graphFormat())
-);

applications/utilities/postProcessing/miscellaneous/pdfPlot/pdfPlot.C

@@ -75,7 +75,7 @@ int main(int argc, char *argv[])
         x[i] = xMin + i*(xMax - xMin)/(nIntervals - 1);
     }
 
-    setWriter::New(runTime.graphFormat())->write
+    setWriter::New(runTime.controlDict().lookup("graphFormat"))->write
     (
         pdfPath,
         args.executable(),

applications/utilities/postProcessing/noise/noise.C

@@ -155,7 +155,7 @@ int main(int argc, char *argv[])
     // Graph format
     const word graphFormat
     (
-        dict.lookupOrDefault<word>("graphFormat", runTime.graphFormat())
+        dict.lookupOrDefault<word>("graphFormat", "raw")
     );
 
     Info<< "Reading data file" << endl;

src/OpenFOAM/db/Time/Time.C

@@ -385,7 +385,6 @@ Foam::Time::Time
     writeFormat_(IOstream::ASCII),
     writeVersion_(IOstream::currentVersion),
     writeCompression_(IOstream::UNCOMPRESSED),
-    graphFormat_("raw"),
     cacheTemporaryObjects_(true),
 
     functionObjects_
@@ -484,7 +483,6 @@ Foam::Time::Time
     writeFormat_(IOstream::ASCII),
     writeVersion_(IOstream::currentVersion),
     writeCompression_(IOstream::UNCOMPRESSED),
-    graphFormat_("raw"),
     cacheTemporaryObjects_(true),
 
     functionObjects_(*this, enableFunctionObjects)
@@ -550,7 +548,6 @@ Foam::Time::Time
     writeFormat_(IOstream::ASCII),
     writeVersion_(IOstream::currentVersion),
     writeCompression_(IOstream::UNCOMPRESSED),
-    graphFormat_("raw"),
     cacheTemporaryObjects_(true),
 
     functionObjects_(*this, enableFunctionObjects)
@@ -611,7 +608,6 @@ Foam::Time::Time
     writeFormat_(IOstream::ASCII),
     writeVersion_(IOstream::currentVersion),
     writeCompression_(IOstream::UNCOMPRESSED),
-    graphFormat_("raw"),
     cacheTemporaryObjects_(true),
 
     functionObjects_(*this, enableFunctionObjects)

src/OpenFOAM/db/Time/Time.H

@@ -190,9 +190,6 @@ private:
         //- Default output compression
         IOstream::compressionType writeCompression_;
 
-        //- Default graph format
-        word graphFormat_;
-
         //- Is temporary object cache enabled
         mutable bool cacheTemporaryObjects_;
 
@@ -301,12 +298,6 @@ public:
                 return writeCompression_;
             }
 
-            //- Default graph format
-            const word& graphFormat() const
-            {
-                return graphFormat_;
-            }
-
             //- Supports re-reading
             const Switch& runTimeModifiable() const
             {

src/OpenFOAM/db/Time/TimeIO.C

@@ -2,7 +2,7 @@
   =========                 |
   \\      /  F ield         | OpenFOAM: The Open Source CFD Toolbox
    \\    /   O peration     | Website:  https://openfoam.org
-    \\  /    A nd           | Copyright (C) 2011-2022 OpenFOAM Foundation
+    \\  /    A nd           | Copyright (C) 2011-2023 OpenFOAM Foundation
      \\/     M anipulation  |
 -------------------------------------------------------------------------------
 License
@@ -202,7 +202,6 @@ void Foam::Time::readDict()
         }
     }
 
-    controlDict_.readIfPresent("graphFormat", graphFormat_);
     controlDict_.readIfPresent("runTimeModifiable", runTimeModifiable_);
 
     userTime_->read(controlDict_);

src/functionObjects/field/histogram/histogram.C

@@ -71,11 +71,7 @@ bool Foam::functionObjects::histogram::read(const dictionary& dict)
     min_ = dict.lookupOrDefault<scalar>("min", 0);
     dict.lookup("nBins") >> nBins_;
 
-    formatterPtr_ = setWriter::New
+    formatterPtr_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     return true;
 }

src/functionObjects/field/layerAverage/layerAverage.C

@@ -245,11 +245,7 @@ bool Foam::functionObjects::layerAverage::read(const dictionary& dict)
 
     fields_ = dict.lookup<wordList>("fields");
 
-    formatter_ = setWriter::New
+    formatter_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     calcLayers();
 

src/functionObjects/field/patchCutLayerAverage/patchCutLayerAverage.C

@@ -658,11 +658,7 @@ bool Foam::functionObjects::patchCutLayerAverage::read(const dictionary& dict)
             )
         ];
 
-    formatter_ = setWriter::New
+    formatter_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     nOptimiseIter_ = dict.lookupOrDefault("nOptimiseIter", 2);
 

src/functionObjects/field/regionSizeDistribution/regionSizeDistribution.C

@@ -371,11 +371,7 @@ bool Foam::functionObjects::regionSizeDistribution::read(const dictionary& dict)
     dict.lookup("nBins") >> nBins_;
     dict.lookup("fields") >> fields_;
 
-    formatterPtr_ = setWriter::New
+    formatterPtr_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     return true;
 }

src/functionObjects/field/streamlines/streamlines.C

@@ -181,11 +181,7 @@ bool Foam::functionObjects::streamlines::read(const dictionary& dict)
         dict.subDict("seedSampleSet")
     );
 
-    formatterPtr_ = setWriter::New
+    formatterPtr_ = setWriter::New(dict.lookup("setFormat"), dict);
-    (
-        dict.lookupOrDefault("setFormat", time_.graphFormat()),
-        dict
-    );
 
     return true;
 }

src/lagrangian/parcel/submodels/CloudFunctionObjects/SizeDistribution/SizeDistribution.C

@@ -108,18 +108,7 @@ Foam::SizeDistribution<CloudType>::SizeDistribution
 :
     CloudFunctionObject<CloudType>(dict, owner, modelName, typeName),
     nPoints_(dict.lookup<label>("nPoints")),
-    formatter_
+    formatter_(setWriter::New(dict.lookup("setFormat"), dict))
-    (
-        setWriter::New
-        (
-            dict.lookupOrDefault
-            (
-                "setFormat",
-                owner.mesh().time().graphFormat()
-            ),
-            dict
-        )
-    )
 {}
 
 

src/randomProcesses/fft/writeEk.C

@@ -58,7 +58,7 @@ void writeEk
 
     autoPtr<setWriter> writer
     (
-        setWriter::New(U.time().graphFormat())
+        setWriter::New(U.time().controlDict().lookup("graphFormat"))
     );
 
     writer->write

src/sampling/sampledSet/sampledSets/sampledSets.C

@@ -145,10 +145,7 @@ bool Foam::functionObjects::sampledSets::read(const dictionary& dict)
 
         dict.lookup("interpolationScheme") >> interpolationScheme_;
 
-        const word writeType
+        const word writeType(dict.lookup("setFormat"));
-        (
-            dict.lookupOrDefault("setFormat", time_.graphFormat())
-        );
 
         // Define the set formatter
         formatter_ = setWriter::New(writeType, dict);

tutorials/fluid/angledDuctExplicitFixedCoeff/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/fluid/squareBend/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/fluid/squareBendLiq/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/fluid/squareBendLiqSteady/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/legacy/compressible/rhoPorousSimpleFoam/angledDuctExplicit/system/controlDict

@@ -42,8 +42,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/legacy/compressible/rhoPorousSimpleFoam/angledDuctImplicit/system/controlDict

@@ -42,8 +42,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/legacy/incompressible/dnsFoam/boxTurb16/system/controlDict

@@ -44,7 +44,5 @@ timePrecision   6;
 
 runTimeModifiable true;
 
-graphFormat     raw;
-
 
 // ************************************************************************* //

tutorials/legacy/incompressible/porousSimpleFoam/angledDuctExplicit/system/controlDict

@@ -42,8 +42,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/legacy/incompressible/porousSimpleFoam/angledDuctImplicit/system/controlDict

@@ -42,8 +42,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 

tutorials/multicomponentFluid/smallPoolFire2D/system/controlDict

@@ -43,8 +43,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable yes;
 
 adjustTimeStep  yes;

tutorials/multicomponentFluid/smallPoolFire3D/system/controlDict

@@ -43,8 +43,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable yes;
 
 adjustTimeStep  yes;

tutorials/shockFluid/obliqueShock/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;
 
 adjustTimeStep  yes;

tutorials/solidDisplacement/plateHole/system/controlDict

@@ -44,8 +44,6 @@ timeFormat      general;
 
 timePrecision   6;
 
-graphFormat     raw;
-
 runTimeModifiable true;