Skip to content

Commit

Permalink
Rename model_parser_cxx to strip unnecessary C++ identifier (#2546)
Browse files Browse the repository at this point in the history
  • Loading branch information
maxpkatz authored Sep 8, 2023
1 parent f6adf9d commit ecf66c3
Show file tree
Hide file tree
Showing 34 changed files with 35 additions and 35 deletions.
2 changes: 1 addition & 1 deletion .clang-tidy
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ Checks: >
mpi-*,
openmp-*
HeaderFilterRegex: '(/Source/*/|/Util/model_parser_cxx|^\./|^tmp_build_dir/castro_sources/*/).*\.H$'
HeaderFilterRegex: '(/Source/*/|/Util/model_parser|^\./|^tmp_build_dir/castro_sources/*/).*\.H$'
2 changes: 1 addition & 1 deletion .github/workflows/c-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ jobs:
python3 external/cpp-linter-action/run_on_changed_files.py ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }} \
-ignore-files="amrex|Microphysics" \
-config-file="${GITHUB_WORKSPACE}/.clang-tidy" \
-header-filter='/Source/|/Util/model_parser_cxx|^\./' \
-header-filter='/Source/|/Util/model_parser|^\./' \
-run-linter
- name: Archive clang tidy report
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/good_defines.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ BL_FORT_USE_UPPERCASE
BL_LANG_FORT
BL_LAZY
BL_USE_SETBUF
CXX_MODEL_PARSER
MODEL_PARSER
DIFFUSION
DO_PROBLEM_POST_INIT
DO_PROBLEM_POST_RESTART
Expand Down
2 changes: 1 addition & 1 deletion Docs/source/creating_a_problem.rst
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ Many problem setups begin with a 1-d initial model that is mapped onto
the grid. The ``model_parser.H`` provides the functions that read in
the initial model and map it on the Castro grid. To enable this, add::

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

to the problem ``GNUmakefile``. There are 2 other parameters that can
be set in the makefile to control the initial model storage:
Expand Down
6 changes: 3 additions & 3 deletions Exec/Make.Castro
Original file line number Diff line number Diff line change
Expand Up @@ -161,9 +161,9 @@ endif
MAX_NPTS_MODEL ?= 10000
NUM_MODELS ?= 1

ifeq ($(USE_CXX_MODEL_PARSER), TRUE)
Bdirs += Util/model_parser_cxx
DEFINES += -DCXX_MODEL_PARSER -DNPTS_MODEL=$(MAX_NPTS_MODEL) -DNUM_MODELS=$(NUM_MODELS)
ifeq ($(USE_MODEL_PARSER), TRUE)
Bdirs += Util/model_parser
DEFINES += -DMODEL_PARSER -DNPTS_MODEL=$(MAX_NPTS_MODEL) -DNUM_MODELS=$(NUM_MODELS)
endif

# add / define any special physics we need
Expand Down
2 changes: 1 addition & 1 deletion Exec/gravity_tests/StarGrav/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ USE_MPI = TRUE
USE_GRAV = TRUE
USE_REACT = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
EOS_DIR := helmholtz
Expand Down
2 changes: 1 addition & 1 deletion Exec/gravity_tests/advecting_white_dwarf/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ DIM ?= 3

USE_GRAV ?= TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE
NUM_MODELS = 1

COMP ?= gnu
Expand Down
2 changes: 1 addition & 1 deletion Exec/gravity_tests/hse_convergence/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ USE_MPI = TRUE
USE_GRAV = TRUE
USE_REACT = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

USE_MAESTRO_INIT = FALSE

Expand Down
2 changes: 1 addition & 1 deletion Exec/gravity_tests/hse_convergence_general/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ USE_REACT = FALSE
USE_ROTATION = FALSE
USE_DIFFUSION = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# define the location of the CASTRO top directory
CASTRO_HOME := ../../..
Expand Down
2 changes: 1 addition & 1 deletion Exec/gravity_tests/hydrostatic_adjust/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ USE_MPI = TRUE

USE_GRAV = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/hydro_tests/double_bubble/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ COMP = gnu
USE_MPI = TRUE
USE_GRAV = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# define the location of the CASTRO top directory
CASTRO_HOME := ../../..
Expand Down
2 changes: 1 addition & 1 deletion Exec/hydro_tests/gamma_law_bubble/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ COMP = gnu
USE_MPI = TRUE
USE_GRAV = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

USE_MAESTRO_INIT = FALSE

Expand Down
2 changes: 1 addition & 1 deletion Exec/hydro_tests/test_convect/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ USE_OMP = FALSE
USE_GRAV = TRUE
USE_REACT = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
EOS_DIR := helmholtz
Expand Down
2 changes: 1 addition & 1 deletion Exec/hydro_tests/toy_convect/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ USE_OMP = FALSE
USE_GRAV = TRUE
USE_REACT = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/radiation_tests/RadBreakout/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ USE_GRAV = TRUE

USE_RAD = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/reacting_tests/bubble_convergence/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ USE_MPI = TRUE
USE_GRAV = TRUE
USE_REACT = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

USE_MAESTRO_INIT = FALSE

Expand Down
2 changes: 1 addition & 1 deletion Exec/reacting_tests/reacting_bubble/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ USE_MPI = TRUE
USE_GRAV = TRUE
USE_REACT = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE
USE_MAESTRO_INIT = FALSE

CASTRO_HOME = ../../..
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/bwp-rad/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ USE_RAD = TRUE
USE_GRAV = TRUE
USE_REACT = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/science/celldet/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ USE_MHD = FALSE

#USE_PROB_PARAMS = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/science/convective_flame/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ USE_REACT = TRUE
USE_ROTATION = TRUE
USE_DIFFUSION = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE
NUM_MODELS := 2

CASTRO_HOME = ../../..
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/flame_wave/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ USE_DIFFUSION = TRUE
CASTRO_HOME := ../../..

USE_JACOBIAN_CACHING = TRUE
USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE
NUM_MODELS := 2

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/massive_star/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ USE_NSE_TABLE = TRUE

BL_NO_FORT := TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
EOS_DIR := helmholtz
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/nova/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ USE_GRAV = TRUE

CASTRO_HOME := ../../..

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
EOS_DIR := helmholtz
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/planet/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ USE_PARTICLE = FALSE
USE_SHOCK_VAR = TRUE
USE_ROTATION = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/science/subchandra/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ USE_MPI = TRUE
USE_GRAV = TRUE
USE_REACT = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

USE_SIMPLIFIED_SDC=TRUE

Expand Down
2 changes: 1 addition & 1 deletion Exec/science/subchandra/GNUmakefile.nse_net
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ USE_REACT = TRUE

USE_NSE_NET = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

# This sets the EOS directory in $(MICROPHYSICS_HOME)/eos
EOS_DIR := helmholtz
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/wdmerger/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ USE_GRAV ?= TRUE
USE_REACT ?= TRUE
USE_ROTATION ?= TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE
NUM_MODELS = 2

# define the location of the CASTRO top directory
Expand Down
2 changes: 1 addition & 1 deletion Exec/science/xrb_mixed/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ USE_OMP = FALSE
USE_GRAV = TRUE
USE_REACT = TRUE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

CASTRO_HOME = ../../..

Expand Down
2 changes: 1 addition & 1 deletion Exec/unit_tests/model_burner/GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ USE_RAD = FALSE
USE_PARTICLES = FALSE
USE_ROTATION = FALSE

USE_CXX_MODEL_PARSER = TRUE
USE_MODEL_PARSER = TRUE

USE_REACT = TRUE

Expand Down
8 changes: 4 additions & 4 deletions Source/reactions/Castro_react.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
#include <Castro.H>
#include <Castro_F.H>
#include <advection_util.H>
#ifdef CXX_MODEL_PARSER
#ifdef MODEL_PARSER
#include <model_parser.H>
#endif

Expand Down Expand Up @@ -204,7 +204,7 @@ Castro::react_state(MultiFab& s, MultiFab& r, Real time, Real dt, const int stra
auto mask = mask_covered_zones ? mask_mf.array(mfi) : Array4<Real>{};

const auto dx = geom.CellSizeArray();
#ifdef CXX_MODEL_PARSER
#ifdef MODEL_PARSER
const auto problo = geom.ProbLoArray();
#endif

Expand Down Expand Up @@ -262,7 +262,7 @@ Castro::react_state(MultiFab& s, MultiFab& r, Real time, Real dt, const int stra

burn_state.T_fixed = -1.e30_rt;

#ifdef CXX_MODEL_PARSER
#ifdef MODEL_PARSER
if (drive_initial_convection) {
Real rr[3] = {0.0_rt};

Expand Down Expand Up @@ -604,7 +604,7 @@ Castro::react_state(Real time, Real dt)

burn_state.T_fixed = -1.e30_rt;

#ifdef CXX_MODEL_PARSER
#ifdef MODEL_PARSER
if (drive_initial_convection) {
Real rr[3] = {0.0_rt};

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 comments on commit ecf66c3

Please sign in to comment.