Merge `FillBoundary` and `NodalSync` (`OneStep_nosub`, `OneStep_sub1`) (#3013)

* Add argument for synchronizing nodal points

* Add the option to overridesync in FillBoundaryE / FillBoundaryB

* Remove explicit synchronization

* Cleanup calls to nodal synchronization

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Improve const Correctness

* Cleaning

* Merge NodalSync and FillBoundary First

Restarting the implementation from scratch.
To-do: remove redundant FillBoundary calls.

* Merge Only for OneStep_nosub and OneStep_sub1

* Fix Checksums of Python_wrappers

* Define sync_nodal_points as WarpX Private Member Variable

* Keep Calls to NodalSync for B

* Keep Calls to NodalSync for F

* Reset Benchmark of reduced_diags_single_precision

* OneStep_sub1: Remove All Calls to NodalSync

* OneStep_nosub: Remove All Calls to NodalSync

* Reset Benchmark of divb_cleaning_3d

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Edoardo Zoni <ezoni@lbl.gov>

1 files changed, 2 insertions, 1 deletions

diff --git a/Source/Parallelization/WarpXCommUtil.H b/Source/Parallelization/WarpXCommUtil.H
index 2a5ec5390..09d102a56 100644
--- a/Source/Parallelization/WarpXCommUtil.H
+++ b/Source/Parallelization/WarpXCommUtil.H
@@ -59,7 +59,8 @@ void FillBoundary (amrex::MultiFab&          mf,
 
 void FillBoundary (amrex::MultiFab&          mf,
                    amrex::IntVect            ng,
-                   const amrex::Periodicity& period = amrex::Periodicity::NonPeriodic());
+                   const amrex::Periodicity& period = amrex::Periodicity::NonPeriodic(),
+                   const bool                nodal_sync = false);
 
 void FillBoundary (amrex::iMultiFab&         mf,
                    const amrex::Periodicity& period = amrex::Periodicity::NonPeriodic());