From 6c8122d437f9210e05857e4590717de1e87ad50e Mon Sep 17 00:00:00 2001 From: Weiqun Zhang Date: Thu, 28 Sep 2023 08:39:57 -0700 Subject: [PATCH] Code Spell (#3563) Add a github action and fix various spellings. --------- Co-authored-by: Jean M. Sexton --- .codespell-ignore-words | 33 +++++++++++++++++++ .codespellrc | 3 ++ .github/workflows/cleanup-cache-postpr.yml | 4 +-- .github/workflows/cleanup-cache.yml | 4 +-- .github/workflows/codespell.yml | 23 +++++++++++++ Docs/sphinx_documentation/source/AmrCore.rst | 2 +- Src/AmrCore/AMReX_FillPatcher.H | 6 ++-- Src/AmrCore/AMReX_FluxRegister.H | 6 ++-- Src/Base/AMReX.cpp | 6 ++-- Src/Base/AMReX_ANSIEscCode.H | 2 +- Src/Base/AMReX_Arena.H | 2 +- Src/Base/AMReX_BLProfiler.cpp | 2 +- Src/Base/AMReX_CArena.H | 2 +- Src/Base/AMReX_CTOParallelForImpl.H | 2 +- Src/Base/AMReX_DistributionMapping.H | 2 +- Src/Base/AMReX_FabArray.H | 6 ++-- Src/Base/AMReX_FabArrayUtility.H | 2 +- Src/Base/AMReX_GpuControl.H | 4 +-- Src/Base/AMReX_IntVect.H | 2 +- Src/Base/AMReX_MultiFabUtilI.H | 2 +- Src/Base/AMReX_PODVector.H | 2 +- Src/Base/AMReX_ParReduce.H | 18 +++++----- Src/Base/AMReX_Utility.cpp | 2 +- Src/Base/AMReX_VisMF.H | 2 +- Src/Boundary/AMReX_InterpBndryData.H | 2 +- Src/Boundary/AMReX_YAFluxRegister.H | 2 +- Src/EB/AMReX_EB2_GeometryShop.H | 2 +- Src/EB/AMReX_EB2_Level.H | 2 +- Src/EB/AMReX_EB_STL_utils.cpp | 2 +- Src/EB/AMReX_EB_utils.H | 4 +-- Src/Extern/HDF5/AMReX_ParticlesHDF5.H | 4 +-- .../SENSEI/AMReX_AmrMeshDataAdaptor.cpp | 10 +++--- .../AMReX_AmrMeshParticleInSituBridge.H | 2 +- Src/Extern/SUNDIALS/AMReX_Sundials_Core.cpp | 2 +- Src/LinearSolvers/MLMG/AMReX_MLLinOp.H | 8 ++--- .../MLMG/AMReX_MLNodeLaplacian.cpp | 4 +-- Src/Particle/AMReX_DenseBins.H | 12 +++---- Src/Particle/AMReX_NeighborParticles.H | 2 +- Src/Particle/AMReX_ParticleCommunication.H | 2 +- Src/Particle/AMReX_ParticleContainer.H | 8 ++--- .../Advection_F/Source/my_amr_mod.F90 | 6 ++-- .../Advection_octree_F/Source/my_amr_mod.F90 | 6 ++-- Tests/GPU/Vector/main.cpp | 4 +-- Tests/HDF5Benchmark/sz.config | 2 +- Tests/LinearSolvers/Hypre/MyTest.cpp | 4 +-- Tests/Parser2/main.cpp | 2 +- Tests/Particles/GhostsAndVirtuals/main.cpp | 2 +- Tools/CMake/AMReXConfig.cmake.in | 2 +- Tools/CMake/AMReXOptions.cmake | 2 +- Tools/C_scripts/mmclt.py | 4 +-- Tools/GNUMake/Make.defs | 2 +- Tools/GNUMake/Make.rules | 2 +- Tools/GNUMake/comps/hip.mak | 2 +- 53 files changed, 153 insertions(+), 94 deletions(-) create mode 100644 .codespell-ignore-words create mode 100644 .codespellrc create mode 100644 .github/workflows/codespell.yml diff --git a/.codespell-ignore-words b/.codespell-ignore-words new file mode 100644 index 00000000000..4e265268e63 --- /dev/null +++ b/.codespell-ignore-words @@ -0,0 +1,33 @@ +abot +alo +apoints +asend +ba +bloc +blocs +boxs +cant +ccache +clen +compex +couldnt +fromm +frop +geometrys +hist +hsi +infor +inout +ist +lsit +nd +nineth +parm +parms +pres +ptd +recuse +siz +structed +te +thi diff --git a/.codespellrc b/.codespellrc new file mode 100644 index 00000000000..7fa3560731f --- /dev/null +++ b/.codespellrc @@ -0,0 +1,3 @@ +[codespell] +skip = .git,*.ipynb,*.bib,*.ps,*.patch,*~,CHANGES,*/Extern/SWFFT,*/Extern/hpgmg,./tmp_install_dir,./installdir,*/build,*/tmp_build_dir +ignore-words = .codespell-ignore-words diff --git a/.github/workflows/cleanup-cache-postpr.yml b/.github/workflows/cleanup-cache-postpr.yml index a43b89ee8b0..73d6eaf0903 100644 --- a/.github/workflows/cleanup-cache-postpr.yml +++ b/.github/workflows/cleanup-cache-postpr.yml @@ -8,7 +8,7 @@ on: jobs: CleanUpCcacheCachePostPR: - name: Clean Up Ccahe Cache Post PR + name: Clean Up Ccache Cache Post PR runs-on: ubuntu-latest permissions: actions: write @@ -17,7 +17,7 @@ jobs: GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} steps: - uses: actions/checkout@v3 - - name: Clean up ccahe + - name: Clean up ccache run: | gh extension install actions/gh-actions-cache diff --git a/.github/workflows/cleanup-cache.yml b/.github/workflows/cleanup-cache.yml index 4b0f444885b..2a1a2f254a3 100644 --- a/.github/workflows/cleanup-cache.yml +++ b/.github/workflows/cleanup-cache.yml @@ -8,7 +8,7 @@ on: jobs: CleanUpCcacheCache: - name: Clean Up Ccahe Cache for ${{ github.event.workflow_run.name }} + name: Clean Up Ccache Cache for ${{ github.event.workflow_run.name }} runs-on: ubuntu-latest permissions: actions: write @@ -17,7 +17,7 @@ jobs: GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} steps: - uses: actions/checkout@v3 - - name: Clean up ccahe + - name: Clean up ccache run: | gh extension install actions/gh-actions-cache diff --git a/.github/workflows/codespell.yml b/.github/workflows/codespell.yml new file mode 100644 index 00000000000..491b839c00e --- /dev/null +++ b/.github/workflows/codespell.yml @@ -0,0 +1,23 @@ +name: codespell + +on: [push, pull_request] + +concurrency: + group: ${{ github.ref }}-${{ github.head_ref }}-codespell + cancel-in-progress: true + +jobs: + codespell: + runs-on: ubuntu-latest + + steps: + - uses: actions/checkout@v3 + + - name: Install codespell + run: | + sudo apt-get update + sudo apt-get install -y --no-install-recommends python3-pip + pip3 install --user codespell + + - name: Run codespell + run: codespell diff --git a/Docs/sphinx_documentation/source/AmrCore.rst b/Docs/sphinx_documentation/source/AmrCore.rst index b444efffab9..6aadd62250e 100644 --- a/Docs/sphinx_documentation/source/AmrCore.rst +++ b/Docs/sphinx_documentation/source/AmrCore.rst @@ -238,7 +238,7 @@ Within AMReX_Interpolater.cpp/H are the derived classes: - :cpp:`FaceLinear` -- :cpp:`FaceDivFree`: This is more accurately a divergence-preserving interpolation on face centered data, i.e., it ensures the divergence of the fine ghost cells match the value of the divergence of the underlying coarse cell. All fine cells overlying a given coarse cell will have the same divergence, even when the coarse grid divergence is spatially varying. Note that when using this with :cpp:`FillPatch` for time sub-cycling, the coarse grid times may not match the fine grid time, in which case :cpp:`FillPatch` will create coarse values at the fine time before calling this interpolation and the result of the :cpp:`FillPatch` is *not* garanteed to preserve the original divergence. +- :cpp:`FaceDivFree`: This is more accurately a divergence-preserving interpolation on face centered data, i.e., it ensures the divergence of the fine ghost cells match the value of the divergence of the underlying coarse cell. All fine cells overlying a given coarse cell will have the same divergence, even when the coarse grid divergence is spatially varying. Note that when using this with :cpp:`FillPatch` for time sub-cycling, the coarse grid times may not match the fine grid time, in which case :cpp:`FillPatch` will create coarse values at the fine time before calling this interpolation and the result of the :cpp:`FillPatch` is *not* guaranteed to preserve the original divergence. These Interpolaters can be executed on CPU or GPU, with certain limitations: diff --git a/Src/AmrCore/AMReX_FillPatcher.H b/Src/AmrCore/AMReX_FillPatcher.H index abc4597ebae..46d1107dea3 100644 --- a/Src/AmrCore/AMReX_FillPatcher.H +++ b/Src/AmrCore/AMReX_FillPatcher.H @@ -30,7 +30,7 @@ namespace amrex { * (1) This class is for filling data during time stepping, not during * regrid. The fine level data passed as input must have the same BoxArray * and DistributionMapping as the destination. It's OK they are the same - * MultiFab. For AmrLevel based codes, AmrLevel::FillPatcherFill wil try to + * MultiFab. For AmrLevel based codes, AmrLevel::FillPatcherFill will try to * use FillPatcher if it can, and AmrLevel::FillPatch will use the fillpatch * functions. * @@ -64,7 +64,7 @@ namespace amrex { * AMR levels. This operation at the coarse/fine boundary is non-trivial * for RK orders higher than 2. Note that it is expected that time stepping * on the coarse level is perform before any fine level time stepping, and - * it's the user's reponsibility to properly create and destroy this object. + * it's the user's responsibility to properly create and destroy this object. * See AmrLevel::RK for an example of using the RungeKutta functions and * FillPatcher together. */ @@ -419,7 +419,7 @@ void FillPatcher::storeRKCoarseData (Real /*time*/, Real dt, MF const& S_old auto const& fpc = getFPinfo(); for (auto& tmf : m_cf_crse_data) { - tmf.first = std::numeric_limits::lowest(); // because we dont' need it + tmf.first = std::numeric_limits::lowest(); // because we don't need it tmf.second = std::make_unique(make_mf_crse_patch(fpc, m_ncomp)); } m_cf_crse_data[0].second->ParallelCopy(S_old, m_cgeom.periodicity()); diff --git a/Src/AmrCore/AMReX_FluxRegister.H b/Src/AmrCore/AMReX_FluxRegister.H index 95f3cefde26..4178eb289ba 100644 --- a/Src/AmrCore/AMReX_FluxRegister.H +++ b/Src/AmrCore/AMReX_FluxRegister.H @@ -171,7 +171,7 @@ public: const Geometry& geom); /** - * \brief /in this version the area is assumed to muliplied into the flux (if not, use scale to fix) + * \brief /in this version the area is assumed to multiplied into the flux (if not, use scale to fix) * * \param mflx * \param dir @@ -192,7 +192,7 @@ public: /** * \brief Increment flux correction with fine data. * - * /in this version the area is assumed to muliplied into the flux (if not, use scale to fix) + * /in this version the area is assumed to multiplied into the flux (if not, use scale to fix) * * \param mflx * \param dir @@ -230,7 +230,7 @@ public: /** * \brief Increment flux correction with fine data. * - * in this version the area is assumed to muliplied into the flux (if not, use scale to fix) + * in this version the area is assumed to multiplied into the flux (if not, use scale to fix) * * \param flux * \param dir diff --git a/Src/Base/AMReX.cpp b/Src/Base/AMReX.cpp index b43add7c9d1..f0e56952de2 100644 --- a/Src/Base/AMReX.cpp +++ b/Src/Base/AMReX.cpp @@ -408,8 +408,8 @@ amrex::Initialize (int& argc, char**& argv, bool build_parm_parse, else { // This counts command line arguments before a "--" - // and only sends the preceeding arguments to ParmParse; - // the rest get ingored. + // and only sends the preceding arguments to ParmParse; + // the rest get ignored. int ppargc = 1; for (; ppargc < argc; ++ppargc) { if (std::strcmp(argv[ppargc], "--") == 0) { break; } @@ -514,7 +514,7 @@ amrex::Initialize (int& argc, char**& argv, bool build_parm_parse, pp.queryAdd("handle_sigfpe" , system::handle_sigfpe ); pp.queryAdd("handle_sigill" , system::handle_sigill ); - // We save the singal handlers and restore them in Finalize. + // We save the signal handlers and restore them in Finalize. if (system::handle_sigsegv) { prev_handler_sigsegv = std::signal(SIGSEGV, BLBackTrace::handler); } else { diff --git a/Src/Base/AMReX_ANSIEscCode.H b/Src/Base/AMReX_ANSIEscCode.H index 02f4bbc059f..1c348deabf1 100644 --- a/Src/Base/AMReX_ANSIEscCode.H +++ b/Src/Base/AMReX_ANSIEscCode.H @@ -16,7 +16,7 @@ namespace Font { constexpr char RapidBlink [] = "\033[6m"; } -namespace FGColor { // Forground colors +namespace FGColor { // Foreground colors constexpr char Black [] = "\033[30m"; constexpr char Red [] = "\033[31m"; constexpr char Green [] = "\033[32m"; diff --git a/Src/Base/AMReX_Arena.H b/Src/Base/AMReX_Arena.H index c3dad6ac2b4..e42ebdc9cd2 100644 --- a/Src/Base/AMReX_Arena.H +++ b/Src/Base/AMReX_Arena.H @@ -146,7 +146,7 @@ public: /** * \brief Does the device have enough free memory for allocating this * much memory? For CPU builds, this always return true. This is not a - * const funciton because it may attempt to release memory back to the + * const function because it may attempt to release memory back to the * system. */ [[nodiscard]] virtual bool hasFreeDeviceMemory (std::size_t sz); diff --git a/Src/Base/AMReX_BLProfiler.cpp b/Src/Base/AMReX_BLProfiler.cpp index 9d12b23708b..fc94104b7d3 100644 --- a/Src/Base/AMReX_BLProfiler.cpp +++ b/Src/Base/AMReX_BLProfiler.cpp @@ -1281,7 +1281,7 @@ void BLProfiler::WriteCommStats(bool bFlushing, bool memCheck) nfiHeader.Stream() << "CommProfProc " << myProc << " nCommStats " << vCommStats.size() << " datafile " << localDFileName - << " seekpos " << nfiDatafile.SeekPos() // ---- data file seek posotion + << " seekpos " << nfiDatafile.SeekPos() // ---- data file seek position << " " << procName << '\n'; for(int ib(0); ib < CommStats::barrierNames.size(); ++ib) { int seekindex(CommStats::barrierNames[ib].second); diff --git a/Src/Base/AMReX_CArena.H b/Src/Base/AMReX_CArena.H index 8f79c844da8..d68285bc878 100644 --- a/Src/Base/AMReX_CArena.H +++ b/Src/Base/AMReX_CArena.H @@ -70,7 +70,7 @@ public: /** * \brief Does the device have enough free memory for allocating this * much memory? For CPU builds, this always return true. This is not a - * const funciton because it may attempt to release memory back to the + * const function because it may attempt to release memory back to the * system. */ [[nodiscard]] bool hasFreeDeviceMemory (std::size_t sz) final; diff --git a/Src/Base/AMReX_CTOParallelForImpl.H b/Src/Base/AMReX_CTOParallelForImpl.H index 80663728359..73ca3a25e87 100644 --- a/Src/Base/AMReX_CTOParallelForImpl.H +++ b/Src/Base/AMReX_CTOParallelForImpl.H @@ -180,7 +180,7 @@ ParallelFor (TypeList /*list_of_compile_time_options*/, * * \param ctos list of all possible values of the parameters. * \param option the run time parameters. - * \param N an interger specifying the 1D for loop's range. + * \param N an integer specifying the 1D for loop's range. * \param f a callable object taking an integer and working on that iteration. */ template diff --git a/Src/Base/AMReX_DistributionMapping.H b/Src/Base/AMReX_DistributionMapping.H index 2cbd038b5d9..0707532a0fc 100644 --- a/Src/Base/AMReX_DistributionMapping.H +++ b/Src/Base/AMReX_DistributionMapping.H @@ -405,7 +405,7 @@ DistributionMapping MakeSimilarDM (const BoxArray& ba, const MultiFab& mf, const * of overlap with. * * @param[in] ba The BoxArray we want to generate a DistributionMapping for. - * @param[in] src_ba The BoxArray associatied with the src DistributionMapping. + * @param[in] src_ba The BoxArray associated with the src DistributionMapping. * @param[in] src_dm The input DistributionMapping we want the output to be similar to. * @param[in] ng The number of grow cells to use when computing intersection / overlap * @return The computed DistributionMapping. diff --git a/Src/Base/AMReX_FabArray.H b/Src/Base/AMReX_FabArray.H index 59bb6b73807..a8839a4bcc0 100644 --- a/Src/Base/AMReX_FabArray.H +++ b/Src/Base/AMReX_FabArray.H @@ -723,7 +723,7 @@ public: * * \param comp component * \param nghost number of ghost cells - * \param local If true, MPI communciation is skipped. + * \param local If true, MPI communication is skipped. */ template ::value,int> = 0> typename F::value_type @@ -1205,7 +1205,7 @@ public: * \param comp starting component * \param ncomp number of components * \param nghost number of ghost cells - * \param local If true, MPI communciation is skipped. + * \param local If true, MPI communication is skipped. * \param ignore_covered ignore covered cells. Only relevant for cell-centered EB data. */ template ::value,int> = 0> @@ -1220,7 +1220,7 @@ public: * \param comp starting component * \param ncomp number of components * \param nghost number of ghost cells - * \param local If true, MPI communciation is skipped. + * \param local If true, MPI communication is skipped. */ template ::value,int> = 0> typename F::value_type diff --git a/Src/Base/AMReX_FabArrayUtility.H b/Src/Base/AMReX_FabArrayUtility.H index e1e62f08d04..78f3355d34a 100644 --- a/Src/Base/AMReX_FabArrayUtility.H +++ b/Src/Base/AMReX_FabArrayUtility.H @@ -1547,7 +1547,7 @@ indexFromValue (FabArray const& mf, int comp, IntVect const& nghost, * \param ycomp starting component of y * \param ncomp number of components * \param nghost number of ghost cells - * \param local If true, MPI communciation is skipped. + * \param local If true, MPI communication is skipped. */ template ::value,int> FOO = 0> typename FAB::value_type diff --git a/Src/Base/AMReX_GpuControl.H b/Src/Base/AMReX_GpuControl.H index d6f7a55ffaf..0f21213a86d 100644 --- a/Src/Base/AMReX_GpuControl.H +++ b/Src/Base/AMReX_GpuControl.H @@ -150,7 +150,7 @@ namespace Gpu { } /** - * This struct provides a RAII-style mechansim for changing the number + * This struct provides a RAII-style mechanism for changing the number * of streams returned by Gpu::numStreams() to a single stream. */ struct [[nodiscard]] SingleStreamRegion @@ -166,7 +166,7 @@ namespace Gpu { }; /** - * This struct provides a RAII-style mechansim for disabling GPU + * This struct provides a RAII-style mechanism for disabling GPU * synchronization in MFITer by default. Note that explicit calls to * Gpu::steramSynchronize and Gpu::deviceSynchronize still work. */ diff --git a/Src/Base/AMReX_IntVect.H b/Src/Base/AMReX_IntVect.H index 2b8900248a8..fd71c93ae87 100644 --- a/Src/Base/AMReX_IntVect.H +++ b/Src/Base/AMReX_IntVect.H @@ -108,7 +108,7 @@ public: /** * \brief Construct an IntVect from an Vector. It is an error if - * the Vector doesn' t have the same dimension as this + * the Vector doesn't have the same dimension as this * IntVect. */ explicit IntVect (const Vector& a) noexcept : vect{AMREX_D_DECL(a[0],a[1],a[2])} { diff --git a/Src/Base/AMReX_MultiFabUtilI.H b/Src/Base/AMReX_MultiFabUtilI.H index cb7e8892ee1..cae4b86aa3a 100644 --- a/Src/Base/AMReX_MultiFabUtilI.H +++ b/Src/Base/AMReX_MultiFabUtilI.H @@ -64,7 +64,7 @@ namespace amrex::MFUtil { //! Regrid (by duplication) class T (which must be either of type - //! MultiFab or iMultiFab) usign the new BoxArray and + //! MultiFab or iMultiFab) using the new BoxArray and //! DistributionMapping. The boolean flag `regrid_ghost` switched //! between the SymmetricGhost and AsymmetricGhost copy functions. template diff --git a/Src/Base/AMReX_PODVector.H b/Src/Base/AMReX_PODVector.H index 8f71e1e0fd2..0f10dfb94ef 100644 --- a/Src/Base/AMReX_PODVector.H +++ b/Src/Base/AMReX_PODVector.H @@ -422,7 +422,7 @@ namespace amrex iterator insert (const_iterator a_pos, T&& a_item) { - // This is *POD* vector afterall + // This is *POD* vector after all return insert(a_pos, 1, a_item); } diff --git a/Src/Base/AMReX_ParReduce.H b/Src/Base/AMReX_ParReduce.H index 8cbe69345f9..ae5059b2399 100644 --- a/Src/Base/AMReX_ParReduce.H +++ b/Src/Base/AMReX_ParReduce.H @@ -14,7 +14,7 @@ namespace amrex { * the second MultiFab. \verbatim auto const& ma1 = mf1.const_arrays(); - auot const& ma2 = mf2.const_arrays(); + auto const& ma2 = mf2.const_arrays(); GpuTuple mm = ParReduce(TypeList{}, TypeList{}, mf1, mf1.nGrowVect(), @@ -29,7 +29,7 @@ namespace amrex { * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam Ts... data types (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list list of reduce operators * \param type_list list of data types @@ -79,7 +79,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam T data type (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list a reduce operator stored in TypeList * \param type_list a data type stored in TypeList @@ -113,7 +113,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * and the maximum of the second MultiFab. \verbatim auto const& ma1 = mf1.const_arrays(); - auot const& ma2 = mf2.const_arrays(); + auto const& ma2 = mf2.const_arrays(); GpuTuple mm = ParReduce(TypeList{}, TypeList{}, mf1, mf1.nGrowVect(), mf1.nComp(), @@ -128,7 +128,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam Ts... data types (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list list of reduce operators * \param type_list list of data types @@ -175,7 +175,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam T data type (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list a reduce operator stored in TypeList * \param type_list a data type stored in TypeList @@ -210,7 +210,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * the second MultiFab. \verbatim auto const& ma1 = mf1.const_arrays(); - auot const& ma2 = mf2.const_arrays(); + auto const& ma2 = mf2.const_arrays(); GpuTuple mm = ParReduce(TypeList{}, TypeList{}, mf1, [=] AMREX_GPU_DEVICE (int box_no, int i, int j, int k) noexcept @@ -224,7 +224,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam Ts... data types (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list list of reduce operators * \param type_list list of data types @@ -268,7 +268,7 @@ ParReduce (TypeList operation_list, TypeList type_list, * ReduceOpLogicalAnd, and ReduceOpLogicalOr) * \tparam T data type (e.g., Real, int, etc.) * \tparam FAB MultiFab/FabArray type - * \tparam F callable type like a lambda funcion + * \tparam F callable type like a lambda function * * \param operation_list a reduce operator stored in TypeList * \param type_list a data type stored in TypeList diff --git a/Src/Base/AMReX_Utility.cpp b/Src/Base/AMReX_Utility.cpp index 5b84bb3ae6d..16f2f20b554 100644 --- a/Src/Base/AMReX_Utility.cpp +++ b/Src/Base/AMReX_Utility.cpp @@ -193,7 +193,7 @@ amrex::UniqueString() tempstring << std::setprecision(n) << std::fixed << amrex::second(); auto const ts = tempstring.str(); auto const tsl = ts.length(); - return ts.substr(tsl-len,tsl); // tsl-len >= 0 becaues n >= len + return ts.substr(tsl-len,tsl); // tsl-len >= 0 because n >= len } void diff --git a/Src/Base/AMReX_VisMF.H b/Src/Base/AMReX_VisMF.H index fd1473a052d..f0b146f6a9d 100644 --- a/Src/Base/AMReX_VisMF.H +++ b/Src/Base/AMReX_VisMF.H @@ -545,7 +545,7 @@ void read_fab (FAB& fab, VisMF::FabOnDisk const& fod, std::string const& name) * fully defined, the BoxArray on the disk must match the BoxArray in the * given iMultiFab/FabArray object. If it is only constructed * with the default constructor, the BoxArray on the disk will be used and a - * new DistributionMapping will be made. When this funciton is used to + * new DistributionMapping will be made. When this function is used to * restart a calculation from checkpoint files, one should use a fully * defined iMultiFab/FabArray except for the first one in a * series of iMultiFab/MultiFab objects that share the same diff --git a/Src/Boundary/AMReX_InterpBndryData.H b/Src/Boundary/AMReX_InterpBndryData.H index c2b57ba6114..6dabc8ac481 100644 --- a/Src/Boundary/AMReX_InterpBndryData.H +++ b/Src/Boundary/AMReX_InterpBndryData.H @@ -82,7 +82,7 @@ public: * * \param crse BndryRegister storing coarse level data * \param c_start starting component of the coarse data - * \param fine MF containing physical bounadry values + * \param fine MF containing physical boundary values * \param f_start starting component of the fine data * \param bnd_start starting component in this InterpBndryData * \param num_comp number of component diff --git a/Src/Boundary/AMReX_YAFluxRegister.H b/Src/Boundary/AMReX_YAFluxRegister.H index 6f17c4de556..075a630a2f7 100644 --- a/Src/Boundary/AMReX_YAFluxRegister.H +++ b/Src/Boundary/AMReX_YAFluxRegister.H @@ -84,7 +84,7 @@ public: crse_cell = 0, crse_fine_boundary_cell, fine_cell }; - //! For curvilinear cooordinates only. In that case, the flux passed to + //! For curvilinear coordinates only. In that case, the flux passed to //! YAFluxRegister is assumed to have been multiplied by area. Note that //! YAFluxRegister does NOT make a copy of the volume data. So the //! coarse volume MF must be alive during the life time of diff --git a/Src/EB/AMReX_EB2_GeometryShop.H b/Src/EB/AMReX_EB2_GeometryShop.H index 974738e94f1..ee353c13952 100644 --- a/Src/EB/AMReX_EB2_GeometryShop.H +++ b/Src/EB/AMReX_EB2_GeometryShop.H @@ -476,7 +476,7 @@ public: { // interp might still be quiet_nan because lst that // was set to zero has been changed by FillBoundary - // at periodic bounadries. + // at periodic boundaries. inter(i,j,k) = problo[0] + i*dx[0]; } else if (lst(i+1,j,k) == Real(0.0) || diff --git a/Src/EB/AMReX_EB2_Level.H b/Src/EB/AMReX_EB2_Level.H index 6c6f75f784b..cd7f8fc6083 100644 --- a/Src/EB/AMReX_EB2_Level.H +++ b/Src/EB/AMReX_EB2_Level.H @@ -427,7 +427,7 @@ GShopLevel::define_fine (G const& gshop, const Geometry& geom, small_volfrac, geom, extend_domain_face, cover_multiple_cuts, nsm, nmc); - // Becasue it is used in a synchronous reduction kernel in + // Because it is used in a synchronous reduction kernel in // build_cells, we do not need to worry about M2's lifetime. // But we still need to use Elixir to extend the life of // cellflagtmp. diff --git a/Src/EB/AMReX_EB_STL_utils.cpp b/Src/EB/AMReX_EB_STL_utils.cpp index 46a07fc9d9c..b45ef1cad30 100644 --- a/Src/EB/AMReX_EB_STL_utils.cpp +++ b/Src/EB/AMReX_EB_STL_utils.cpp @@ -699,7 +699,7 @@ STLtools::updateIntercept (Array,AMREX_SPACEDIM> const& inter_arr, { // interp might still be quiet_nan because lst that // was set to zero has been changed by FillBoundary - // at periodic bounadries. + // at periodic boundaries. inter(i,j,k) = problo[0] + static_cast(i)*dx[0]; } else if (lst(i+1,j,k) == Real(0.0) || diff --git a/Src/EB/AMReX_EB_utils.H b/Src/EB/AMReX_EB_utils.H index 10c5246c9e6..73c11ac95a3 100644 --- a/Src/EB/AMReX_EB_utils.H +++ b/Src/EB/AMReX_EB_utils.H @@ -50,9 +50,9 @@ namespace amrex { * that the distance is valid only if it's within a few cells to the EB. * * \param mf is a nodal MultiFab. - * \param ls_lev is an EB2::Level obejct with an implicit function. This + * \param ls_lev is an EB2::Level object with an implicit function. This * is at the same level as mf. - * \param eb_fac is an EBFArrayBoxFactory object containing EB informaiton. + * \param eb_fac is an EBFArrayBoxFactory object containing EB information. * \param refratio is the refinement ratio of mf to eb_fac. * \param fluid_has_positive_sign determines the sign of the fluid. */ diff --git a/Src/Extern/HDF5/AMReX_ParticlesHDF5.H b/Src/Extern/HDF5/AMReX_ParticlesHDF5.H index 0e5321f7ace..c72f6e22523 100644 --- a/Src/Extern/HDF5/AMReX_ParticlesHDF5.H +++ b/Src/Extern/HDF5/AMReX_ParticlesHDF5.H @@ -172,7 +172,7 @@ public: /** * \brief This version of WritePlotFile assigns component names, but allows the user to pass - * in a vector of ints that toggle on / off the writing of specfic components. + * in a vector of ints that toggle on / off the writing of specific components. * * \param dir The base directory into which to write (i.e. "plt00000") * \param file The name of the sub-directory for this particle type (i.e. "Tracer") @@ -187,7 +187,7 @@ public: /** * \brief This version of WritePlotFile assigns component names, but allows the user to pass - * in a vector of ints that toggle on / off the writing of specfic components. + * in a vector of ints that toggle on / off the writing of specific components. * * This version also lets you pass in a functor to toggle whether each particle gets output. * diff --git a/Src/Extern/SENSEI/AMReX_AmrMeshDataAdaptor.cpp b/Src/Extern/SENSEI/AMReX_AmrMeshDataAdaptor.cpp index 34b92c1d25d..48cd8012b30 100644 --- a/Src/Extern/SENSEI/AMReX_AmrMeshDataAdaptor.cpp +++ b/Src/Extern/SENSEI/AMReX_AmrMeshDataAdaptor.cpp @@ -405,7 +405,7 @@ int AmrMeshDataAdaptor::GetNumberOfArrays(const std::string &meshName, if (this->Internals->States.size() < 1) { - SENSEI_ERROR("No simualtion data, missing call to SetDataSource?") + SENSEI_ERROR("No simulation data, missing call to SetDataSource?") return -1; } @@ -462,7 +462,7 @@ int AmrMeshDataAdaptor::GetMeshHasGhostCells(const std::string &meshName, int &n if (this->Internals->States.size() < 1) { - SENSEI_ERROR("No simualtion data, missing call to SetDataSource?") + SENSEI_ERROR("No simulation data, missing call to SetDataSource?") return -1; } @@ -489,7 +489,7 @@ int AmrMeshDataAdaptor::GetMesh(const std::string &meshName, if (this->Internals->States.size() < 1) { - SENSEI_ERROR("No simualtion data, missing call to SetDataSource?") + SENSEI_ERROR("No simulation data, missing call to SetDataSource?") return -1; } @@ -631,7 +631,7 @@ int AmrMeshDataAdaptor::AddGhostCellsArray(svtkDataObject* mesh, if (this->Internals->States.size() < 1) { - SENSEI_ERROR("No simualtion data, missing call to SetDataSource?") + SENSEI_ERROR("No simulation data, missing call to SetDataSource?") return -1; } @@ -756,7 +756,7 @@ int AmrMeshDataAdaptor::AddArray(svtkDataObject* mesh, if (this->Internals->States.size() < 1) { - SENSEI_ERROR("No simualtion data, missing call to SetDataSource?") + SENSEI_ERROR("No simulation data, missing call to SetDataSource?") return -1; } diff --git a/Src/Extern/SENSEI/AMReX_AmrMeshParticleInSituBridge.H b/Src/Extern/SENSEI/AMReX_AmrMeshParticleInSituBridge.H index 0a85cb7efa9..b8d9dc112a7 100644 --- a/Src/Extern/SENSEI/AMReX_AmrMeshParticleInSituBridge.H +++ b/Src/Extern/SENSEI/AMReX_AmrMeshParticleInSituBridge.H @@ -21,7 +21,7 @@ namespace amrex { -/// SENSEI bridge code for simulations proccessing both amrex::Amr and amrex::ParticleContainer +/// SENSEI bridge code for simulations processing both amrex::Amr and amrex::ParticleContainer class AmrMeshParticleInSituBridge : public InSituBridge { public: diff --git a/Src/Extern/SUNDIALS/AMReX_Sundials_Core.cpp b/Src/Extern/SUNDIALS/AMReX_Sundials_Core.cpp index 286662058a3..4533a155ccd 100644 --- a/Src/Extern/SUNDIALS/AMReX_Sundials_Core.cpp +++ b/Src/Extern/SUNDIALS/AMReX_Sundials_Core.cpp @@ -14,7 +14,7 @@ void Initialize(int nthreads) { amrex::Print() << "Initializing SUNDIALS with " << nthreads << " threads...\n"; - // Initalize the sundials context + // Initialize the sundials context if (initialized.empty()) { initialized.resize(nthreads); std::fill(initialized.begin(), initialized.end(), 0); diff --git a/Src/LinearSolvers/MLMG/AMReX_MLLinOp.H b/Src/LinearSolvers/MLMG/AMReX_MLLinOp.H index 32164f54657..706fe679d7e 100644 --- a/Src/LinearSolvers/MLMG/AMReX_MLLinOp.H +++ b/Src/LinearSolvers/MLMG/AMReX_MLLinOp.H @@ -130,7 +130,7 @@ public: * * This functions must be called, and must be called before other bc * functions. This version is for single-component solve or when all the - * compoents have the same BC types. + * components have the same BC types. * * \param lobc lower boundaries * \param hibc upper boundaries @@ -262,7 +262,7 @@ public: virtual void restriction (int amrlev, int cmglev, MF& crse, MF& fine) const = 0; /** - * \brief Add interpolated coarse MG levle data to fine MG level data + * \brief Add interpolated coarse MG level data to fine MG level data * * \param amrlev AMR level * \param fmglev fine MG level @@ -355,7 +355,7 @@ public: * \param resid residual * \param x unknown in the residual-correction form * \param b RHS in the residual-correction form - * \param bc_mode Is the BC homegeneous or inhomogeneous? + * \param bc_mode Is the BC homogeneous or inhomogeneous? * \param crse_bc_data optional argument providing BC at coarse/fine boundary. */ virtual void correctionResidual (int amrlev, int mglev, MF& resid, MF& x, const MF& b, @@ -406,7 +406,7 @@ public: //! This is needed for our nodal projection solver virtual void unimposeNeumannBC (int /*amrlev*/, MF& /*rhs*/) const {} - //! Extra terms introduced when we treate inhomogeneous Nuemann BC as homogeneous. + //! Extra terms introduced when we treat inhomogeneous Nuemann BC as homogeneous. virtual void applyInhomogNeumannTerm (int /*amrlev*/, MF& /*rhs*/) const {} //! for overset solver only diff --git a/Src/LinearSolvers/MLMG/AMReX_MLNodeLaplacian.cpp b/Src/LinearSolvers/MLMG/AMReX_MLNodeLaplacian.cpp index 27e0f6c4b62..4a749a1ed08 100644 --- a/Src/LinearSolvers/MLMG/AMReX_MLNodeLaplacian.cpp +++ b/Src/LinearSolvers/MLMG/AMReX_MLNodeLaplacian.cpp @@ -230,7 +230,7 @@ MLNodeLaplacian::getSolvabilityOffset (int amrlev, int mglev, MultiFab const& rh if (m_lobc[0][idim] != LinOpBCType::Neumann && m_lobc[0][idim] != LinOpBCType::inflow) { - nddom.growLo(idim, 10); // so that the test in ParReduce will faill + nddom.growLo(idim, 10); // so that the test in ParReduce will fail } if (m_hibc[0][idim] != LinOpBCType::Neumann && m_hibc[0][idim] != LinOpBCType::inflow) @@ -334,7 +334,7 @@ MLNodeLaplacian::fixSolvabilityByOffset (int amrlev, int mglev, MultiFab& rhs, if (m_lobc[0][idim] != LinOpBCType::Neumann && m_lobc[0][idim] != LinOpBCType::inflow) { - nddom.growLo(idim, 10); // so that the test in ParReduce will faill + nddom.growLo(idim, 10); // so that the test in ParReduce will fail } if (m_hibc[0][idim] != LinOpBCType::Neumann && m_hibc[0][idim] != LinOpBCType::inflow) diff --git a/Src/Particle/AMReX_DenseBins.H b/Src/Particle/AMReX_DenseBins.H index 67ff2c1c9c5..5ff9ecdc23f 100644 --- a/Src/Particle/AMReX_DenseBins.H +++ b/Src/Particle/AMReX_DenseBins.H @@ -111,7 +111,7 @@ public: * Finally, the set of partial sums is incremented in parallel using atomicInc, * which results in a permutation array that places the items in bin-sorted order. * - * This version uses a 3D box to specificy the index space over which the bins + * This version uses a 3D box to specify the index space over which the bins * are defined. * * This overload uses the "default" parallelization strategy. If AMReX has been @@ -166,7 +166,7 @@ public: * Finally, the set of partial sums is incremented in parallel using atomicInc, * which results in a permutation array that places the items in bin-sorted order. * - * This version uses a 3D box to specificy the index space over which the bins + * This version uses a 3D box to specify the index space over which the bins * are defined. * * This overload uses the "GPU" parallelization strategy. If AMReX has been @@ -269,11 +269,11 @@ public: * Finally, the set of partial sums is incremented in parallel using atomicInc, * which results in a permutation array that places the items in bin-sorted order. * - * This version uses a 3D box to specificy the index space over which the bins + * This version uses a 3D box to specify the index space over which the bins * are defined. * * This overload uses the "OpenMP" parallelization strategy and always runs on the - * host. If AMReX has been compiled with OpenMP support, the executation will be + * host. If AMReX has been compiled with OpenMP support, the execution will be * parallelized, otherwise it will be serial. * * \tparam N the 'size' type that can enumerate all the items @@ -315,7 +315,7 @@ public: * This version uses a 1D index space for the set of bins. * * This overload uses the "OpenMP" parallelization strategy and always runs on the - * host. If AMReX has been compiled with OpenMP support, the executation will be + * host. If AMReX has been compiled with OpenMP support, the execution will be * parallelized, otherwise it will be serial. * * \tparam N the 'size' type that can enumerate all the items @@ -408,7 +408,7 @@ public: * Finally, the set of partial sums is incremented in parallel using atomicInc, * which results in a permutation array that places the items in bin-sorted order. * - * This version uses a 3D box to specificy the index space over which the bins + * This version uses a 3D box to specify the index space over which the bins * are defined. * * This overload uses the "Serial" parallelization strategy. It always runs in diff --git a/Src/Particle/AMReX_NeighborParticles.H b/Src/Particle/AMReX_NeighborParticles.H index 5789f6f206d..781c9ddc446 100644 --- a/Src/Particle/AMReX_NeighborParticles.H +++ b/Src/Particle/AMReX_NeighborParticles.H @@ -19,7 +19,7 @@ namespace amrex { }; /// -/// This is a container for particles that undergo short-range interations. +/// This is a container for particles that undergo short-range interactions. /// In addition to the normal ParticleContainer methods, each tile contains a "neighbor /// buffer" that is filled with data corresponding to the particles within 1 neighbor cell of /// the tile boundaries. This allows the N^2 search over each pair of particles to proceed diff --git a/Src/Particle/AMReX_ParticleCommunication.H b/Src/Particle/AMReX_ParticleCommunication.H index 36ecb747e4b..ba9a4faba49 100644 --- a/Src/Particle/AMReX_ParticleCommunication.H +++ b/Src/Particle/AMReX_ParticleCommunication.H @@ -237,7 +237,7 @@ private: // // Snds - a Vector with the number of bytes that is process will send to each proc. // Rcvs - a Vector that, after calling this method, will contain the - // number of bytes this process will reveive from each proc. + // number of bytes this process will receive from each proc. // void doHandShake (const Vector& Snds, Vector& Rcvs) const; diff --git a/Src/Particle/AMReX_ParticleContainer.H b/Src/Particle/AMReX_ParticleContainer.H index 66897cc09f9..df71f2eeec1 100644 --- a/Src/Particle/AMReX_ParticleContainer.H +++ b/Src/Particle/AMReX_ParticleContainer.H @@ -421,7 +421,7 @@ public: /** * \brief * This initializes the particle container with one particle per cell, - * where the other particle data and attributes are all contant. The + * where the other particle data and attributes are all constant. The * coarsest level is used to generate the particle positions. The particle * variable values are passed in through the pdata struct. The parameters * x_off, y_off, and z_off represent offsets between 0 and 1 that show @@ -750,7 +750,7 @@ public: /** * \brief Writes a particle checkpoint to file, suitable for restarting. * This version allows the particle component names to be passed in. - * This overload exists for backwards comptability. The is_checkpoint parameter is ignored. + * This overload exists for backwards compatibility. The is_checkpoint parameter is ignored. */ void Checkpoint (const std::string& dir, const std::string& name, bool is_checkpoint, const Vector& real_comp_names = Vector(), @@ -968,13 +968,13 @@ public: void WriteAsciiFile (const std::string& file); /** - * \brief Return the underyling Vector (over AMR levels) of ParticleLevels. + * \brief Return the underlying Vector (over AMR levels) of ParticleLevels. * Const version. */ const Vector& GetParticles () const { return m_particles; } /** - * \brief Return the underyling Vector (over AMR levels) of ParticleLevels. + * \brief Return the underlying Vector (over AMR levels) of ParticleLevels. * Non-const version. */ Vector & GetParticles () { return m_particles; } diff --git a/Tests/FortranInterface/Advection_F/Source/my_amr_mod.F90 b/Tests/FortranInterface/Advection_F/Source/my_amr_mod.F90 index 8df5117d77e..e9549d57320 100644 --- a/Tests/FortranInterface/Advection_F/Source/my_amr_mod.F90 +++ b/Tests/FortranInterface/Advection_F/Source/my_amr_mod.F90 @@ -104,7 +104,7 @@ subroutine my_amr_finalize () end subroutine my_amr_finalize ! Make a new level from scratch and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_make_new_level_from_scratch (lev, time, pba, pdm) bind(c) use prob_module, only : init_prob_data integer, intent(in), value :: lev @@ -145,7 +145,7 @@ subroutine my_make_new_level_from_scratch (lev, time, pba, pdm) bind(c) end subroutine my_make_new_level_from_scratch ! Make a new level from coarse level and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_make_new_level_from_coarse (lev, time, pba, pdm) bind(c) use fillpatch_module, only : fillcoarsepatch integer, intent(in), value :: lev @@ -173,7 +173,7 @@ subroutine my_make_new_level_from_coarse (lev, time, pba, pdm) bind(c) end subroutine my_make_new_level_from_coarse ! Remake a level from current and coarse elvels and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_remake_level (lev, time, pba, pdm) bind(c) use fillpatch_module, only : fillpatch integer, intent(in), value :: lev diff --git a/Tests/FortranInterface/Advection_octree_F/Source/my_amr_mod.F90 b/Tests/FortranInterface/Advection_octree_F/Source/my_amr_mod.F90 index ea6e4513ff8..35614b825fa 100644 --- a/Tests/FortranInterface/Advection_octree_F/Source/my_amr_mod.F90 +++ b/Tests/FortranInterface/Advection_octree_F/Source/my_amr_mod.F90 @@ -84,7 +84,7 @@ subroutine my_amr_finalize () end subroutine my_amr_finalize ! Make a new level from scratch and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_make_new_level_from_scratch (lev, time, pba, pdm) bind(c) use prob_module, only : init_prob_data integer, intent(in), value :: lev @@ -127,7 +127,7 @@ subroutine my_make_new_level_from_scratch (lev, time, pba, pdm) bind(c) end subroutine my_make_new_level_from_scratch ! Make a new level from coarse level and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_make_new_level_from_coarse (lev, time, pba, pdm) bind(c) use fillpatch_module, only : fillcoarsepatch integer, intent(in), value :: lev @@ -157,7 +157,7 @@ subroutine my_make_new_level_from_coarse (lev, time, pba, pdm) bind(c) end subroutine my_make_new_level_from_coarse ! Remake a level from current and coarse elvels and put the data in phi_new. - ! Note tha phi_old contains no valid data after this. + ! Note that phi_old contains no valid data after this. subroutine my_remake_level (lev, time, pba, pdm) bind(c) use fillpatch_module, only : fillpatch integer, intent(in), value :: lev diff --git a/Tests/GPU/Vector/main.cpp b/Tests/GPU/Vector/main.cpp index cfa44437fb5..4a3fd1d9583 100644 --- a/Tests/GPU/Vector/main.cpp +++ b/Tests/GPU/Vector/main.cpp @@ -89,9 +89,9 @@ void async_test() // Async Vector now out of scope. Still completes correctly. #ifdef AMREX_USE_GPU - amrex::Print() << "Async Synching -- should print first." << std::endl; + amrex::Print() << "Async Syncing -- should print first." << std::endl; #else - amrex::Print() << "Async Synching -- should print second." << std::endl; + amrex::Print() << "Async Syncing -- should print second." << std::endl; #endif Gpu::Device::streamSynchronize(); diff --git a/Tests/HDF5Benchmark/sz.config b/Tests/HDF5Benchmark/sz.config index 471e9766295..dd363085210 100644 --- a/Tests/HDF5Benchmark/sz.config +++ b/Tests/HDF5Benchmark/sz.config @@ -14,7 +14,7 @@ sol_name = SZ #Examples: layers = 1, 2, or 3 (layers=1 is recommended in most cases) layers = 1 -#sampleDistance determins the number of samples used to optimize the # quantization intervals +#sampleDistance determines the number of samples used to optimize the # quantization intervals #For example, sampleDistance=50 means 1/50=2% of data points are sample points. sampleDistance = 100 diff --git a/Tests/LinearSolvers/Hypre/MyTest.cpp b/Tests/LinearSolvers/Hypre/MyTest.cpp index f68595ffc3f..99290987eeb 100644 --- a/Tests/LinearSolvers/Hypre/MyTest.cpp +++ b/Tests/LinearSolvers/Hypre/MyTest.cpp @@ -25,7 +25,7 @@ MyTest::MyTest () void MyTest::solve () { - // In this example, we assume the domain boundary is homegeneous Dirichlet. + // In this example, we assume the domain boundary is homogeneous Dirichlet. auto const& nddom = amrex::surroundingNodes(geom.Domain()); const auto dxi = geom.InvCellSizeArray(); @@ -51,7 +51,7 @@ MyTest::solve () // [in ] gid : gid[n] is the id for variable n at (i,j,k) // [out] ncols: # of columns in this row. // [out] cols: column indices in this row. - // [out] mat : matrix elemens in this row. + // [out] mat : matrix elements in this row. auto filler = [=] AMREX_GPU_DEVICE (int /*boxno*/, int i, int j, int k, int n, Array4 const* gid, HYPRE_Int& ncols, HYPRE_Int* cols, diff --git a/Tests/Parser2/main.cpp b/Tests/Parser2/main.cpp index dbb1526a20b..70d24bae07e 100644 --- a/Tests/Parser2/main.cpp +++ b/Tests/Parser2/main.cpp @@ -53,7 +53,7 @@ int main (int argc, char* argv[]) failed_tests.push_back(std::make_pair(icase,expr)); } } else { - amrex::Abort("How did this happend? No line after case."); + amrex::Abort("How did this happen? No line after case."); } } } diff --git a/Tests/Particles/GhostsAndVirtuals/main.cpp b/Tests/Particles/GhostsAndVirtuals/main.cpp index 02384eb43ea..1134e3f5b9a 100644 --- a/Tests/Particles/GhostsAndVirtuals/main.cpp +++ b/Tests/Particles/GhostsAndVirtuals/main.cpp @@ -232,7 +232,7 @@ void test_ghosts_and_virtuals_ascii (TestParams& parms) //Initialize particles from ascii file with extradata=4 myPC.InitFromAsciiFile("particle_file.init", 4); - //Regrid to the more compilcated BoxArray similar to NeighborParticleContainer Regrid + //Regrid to the more complicated BoxArray similar to NeighborParticleContainer Regrid for (int lev = 0; lev < nlevs; lev++) { myPC.SetParticleBoxArray(lev, ba.at(lev)); myPC.SetParticleDistributionMap(lev, dmap.at(lev)); diff --git a/Tools/CMake/AMReXConfig.cmake.in b/Tools/CMake/AMReXConfig.cmake.in index b08e0fb162e..7b7f04727a2 100644 --- a/Tools/CMake/AMReXConfig.cmake.in +++ b/Tools/CMake/AMReXConfig.cmake.in @@ -236,7 +236,7 @@ endif () # CMake targets include( "${CMAKE_CURRENT_LIST_DIR}/AMReXTargets.cmake" ) -# CMake targets aliase: last dimension built will be our legacy target +# CMake targets aliases: last dimension built will be our legacy target if (NOT TARGET AMReX::amrex) # protection in case of multiple inclusions list(LENGTH AMReX_SPACEDIM list_len) math(EXPR list_last "${list_len} - 1") diff --git a/Tools/CMake/AMReXOptions.cmake b/Tools/CMake/AMReXOptions.cmake index c086bdc7489..80196639984 100644 --- a/Tools/CMake/AMReXOptions.cmake +++ b/Tools/CMake/AMReXOptions.cmake @@ -59,7 +59,7 @@ print_option( USE_XSDK_DEFAULTS ) # Defaults: # CMake: static # xSDK: shared -# Precendence of user options: +# Precedence of user options: # AMReX_BUILD_SHARED_LIBS > BUILD_SHARED_LIBS > USE_XSDK_DEFAULTS # default: unset unset OFF # diff --git a/Tools/C_scripts/mmclt.py b/Tools/C_scripts/mmclt.py index c8e61ec1b3a..0ddcc0f0c6e 100755 --- a/Tools/C_scripts/mmclt.py +++ b/Tools/C_scripts/mmclt.py @@ -15,9 +15,9 @@ def mmclt(argv): help="Ccache log file", default="ccache.log.txt") parser.add_argument("--identifier", - help="Unique identifier for finding compilaton line in the log file", + help="Unique identifier for finding compilation line in the log file", default="Src/Base") - # We assume Src/Base can be used as an indentifier to distinguish amrex code + # We assume Src/Base can be used as an identifier to distinguish amrex code # from cmake's temporary files like build/CMakeFiles/CMakeScratch/TryCompile-hw3x4m/test_mpi.cpp parser.add_argument("--output", help="Make file for clang-tidy", diff --git a/Tools/GNUMake/Make.defs b/Tools/GNUMake/Make.defs index c1b5eca6bff..7b2ba129b83 100644 --- a/Tools/GNUMake/Make.defs +++ b/Tools/GNUMake/Make.defs @@ -1,6 +1,6 @@ ifneq (,$(findstring ~,$(AMREX_HOME))) - $(warning *** AMREX_HOME string constains ~ and make will not like it. So it is replaced.) + $(warning *** AMREX_HOME string contains ~ and make will not like it. So it is replaced.) AMREX_HOME := $(shell echo $(AMREX_HOME)) endif diff --git a/Tools/GNUMake/Make.rules b/Tools/GNUMake/Make.rules index 53ff408db54..2dba44dca6f 100644 --- a/Tools/GNUMake/Make.rules +++ b/Tools/GNUMake/Make.rules @@ -527,7 +527,7 @@ print-%: @echo ' value = $(subst ','"'"',$(value $*))' # We need to use subst on the result of $(value) because it contains single # quotes. Shell command echo does not like things like 'x'$(filiter-out)'y', -# because what it sees is 'x', $(filter-out), and 'y'. With the substition, it +# because what it sees is 'x', $(filter-out), and 'y'. With the substitution, it # will see 'x', "'", '$(filter-out)', "'", and 'y', with $(filter-out) inside a # pair of single quotes. diff --git a/Tools/GNUMake/comps/hip.mak b/Tools/GNUMake/comps/hip.mak index 424d2d03aa1..f887b206e4c 100644 --- a/Tools/GNUMake/comps/hip.mak +++ b/Tools/GNUMake/comps/hip.mak @@ -43,7 +43,7 @@ HIPCC_FLAGS += -pthread CXXFLAGS += $(HIPCC_FLAGS) -# add fopenmp targetting the gnu library +# add fopenmp targeting the gnu library ifeq ($(USE_OMP),TRUE) CXXFLAGS += -fopenmp=libgomp CFLAGS += -fopenmp=libgomp