Merge pull request #552 from EPIC-model/fix-resetting-diagnostics

Add missing stop_timer; fix resetting parcel split and merge diagnostics
EPIC-model · Jan 24, 2024 · 5d953cd · 5d953cd
2 parents a55654f + 0807d8e
commit 5d953cd
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/3d/parcels/parcel_diagnostics_netcdf.f90 b/src/3d/parcels/parcel_diagnostics_netcdf.f90
@@ -337,7 +337,12 @@ subroutine write_netcdf_parcel_stats(t)
 
             call start_timer(parcel_stats_io_timer)
 
+            ! reset counters for parcel operations
+            n_parcel_splits = 0
+            n_parcel_merges = 0
+
             if (world%rank /= world%root) then
+                call stop_timer(parcel_stats_io_timer)
                 return
             endif
 
@@ -383,10 +388,6 @@ subroutine write_netcdf_parcel_stats(t)
             ! increment counter
             n_writes = n_writes + 1
 
-            ! reset counters for parcel operations
-            n_parcel_splits = 0
-            n_parcel_merges = 0
-
             call close_netcdf_file(ncid, l_serial=.true.)
 
             call stop_timer(parcel_stats_io_timer)