Check for dirtyRecursive more fine-grained

Author: franzpoeschel

include/openPMD/backend/BaseRecord.hpp

@@ -984,6 +984,11 @@ template <typename T_elem>
 inline void BaseRecord<T_elem>::flush(
     std::string const &name, internal::FlushParams const &flushParams)
 {
+    if (!this->dirtyRecursive())
+    {
+        return;
+    }
+
     if (!this->written() && this->empty() && !this->datasetDefined())
         throw std::runtime_error(
             "A Record can not be written without any contained "

src/Iteration.cpp

@@ -361,10 +361,10 @@ void Iteration::flush(internal::FlushParams const &flushParams)
                 s.setMeshesPath("meshes/");
                 s.flushMeshesPath();
             }
-            meshes.flush(s.meshesPath(), flushParams);
-            for (auto &m : meshes)
+            if (meshes.dirtyRecursive())
             {
-                if (m.second.dirtyRecursive())
+                meshes.flush(s.meshesPath(), flushParams);
+                for (auto &m : meshes)
                 {
                     m.second.flush(m.first, flushParams);
                 }
@@ -382,10 +382,10 @@ void Iteration::flush(internal::FlushParams const &flushParams)
                 s.setParticlesPath("particles/");
                 s.flushParticlesPath();
             }
-            particles.flush(s.particlesPath(), flushParams);
-            for (auto &species : particles)
+            if (particles.dirtyRecursive())
             {
-                if (species.second.dirtyRecursive())
+                particles.flush(s.particlesPath(), flushParams);
+                for (auto &species : particles)
                 {
                     species.second.flush(species.first, flushParams);
                 }

src/Mesh.cpp

@@ -366,6 +366,10 @@ template Mesh &Mesh::setTimeOffset(float);
 void Mesh::flush_impl(
     std::string const &name, internal::FlushParams const &flushParams)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     if (access::readOnly(IOHandler()->m_frontendAccess))
     {
         auto &m = get();

src/ParticleSpecies.cpp

@@ -155,6 +155,10 @@ namespace
 void ParticleSpecies::flush(
     std::string const &path, internal::FlushParams const &flushParams)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     if (access::readOnly(IOHandler()->m_frontendAccess))
     {
         for (auto &record : *this)

src/Record.cpp

@@ -52,6 +52,10 @@ Record &Record::setUnitDimension(unit_representations::AsArray const &udim)
 void Record::flush_impl(
     std::string const &name, internal::FlushParams const &flushParams)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     if (access::readOnly(IOHandler()->m_frontendAccess))
     {
         if (scalar())

src/RecordComponent.cpp

@@ -239,6 +239,10 @@ bool RecordComponent::empty() const
 void RecordComponent::flush(
     std::string const &name, internal::FlushParams const &flushParams)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     auto &rc = get();
     if (flushParams.flushLevel == FlushLevel::SkeletonOnly)
     {

src/backend/MeshRecordComponent.cpp

@@ -70,6 +70,10 @@ void MeshRecordComponent::read()
 void MeshRecordComponent::flush(
     std::string const &name, internal::FlushParams const &params)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     if (access::write(IOHandler()->m_frontendAccess) &&
         !containsAttribute("position"))
     {

src/backend/PatchRecord.cpp

@@ -41,6 +41,10 @@ PatchRecord::setUnitDimension(std::map<UnitDimension, double> const &udim)
 void PatchRecord::flush_impl(
     std::string const &path, internal::FlushParams const &flushParams)
 {
+    if (!dirtyRecursive())
+    {
+        return;
+    }
     if (!this->datasetDefined())
     {
         if (IOHandler()->m_frontendAccess != Access::READ_ONLY)