General code comments to be actioned about grdzhadzha

[KAClough]

LinMom -> LinearMomentum (its not so long!)

[KAClough]

• Comments around RHS in Level.cpp classes are out of date (e.g. about setting constraints to zero)
• specificPostTimeStep needs to be better commented

[SamuelBrady]

• ScalarPotential.hpp and ComplexScalarPotential.hpp classes should be moved from Source/Matter to relevant examples

[KAClough]

The TaggingCriterion -> FixedBackgroundTaggingCriterion - just use the one in GRChombo?

[KAClough]

Both examples have a constant field value as the initial conditions - consider making one a gaussian or something more interesting (probably the real SF one)

[KAClough]

Is emtensor_t being duplicated from GRChombo?

[KAClough]

Check header guards as per James. Also copyright should be 2023.

[KAClough]

do_I_excise -> is_excised

[KAClough]

https://github.com/GRChombo/GRChomboFixedBackground/blob/main/Source/Background/BoostedKerrSchildBH.hpp seems to contain BoostedBH class, so need to rename. Paper reference also seems to be wrong - has this been tested? Maybe just remove if not used in the examples.

[KAClough]

Simplify the extraction reading in https://github.com/GRChombo/GRChomboFixedBackground/blob/main/Source/Background/FixedBGSimulationParametersBase.hpp since most params not used - especially for modes etc. which refer to GWs.

[KAClough]

excise returns a double in the function definition but returns a bool in the body.

[KAClough]

we could actually make excise templated over data_t using auto

[KAClough]

The conservation things use the normalised normal vector, but we should be able to remove both this and the dArea (since they cancel out)

[KAClough]

The params for the KerrBH specify symmetry in y and z but it should only be in z

[KAClough]

Missing the python output file for the Kerr example

[KAClough]

There is a problem in the GNUMakeFile in the base directory - GRChombo source is specified but should not be

[KAClough]

The data is not output to the data folder

[KAClough]

The plot of energy conservation has some sign issues - maybe in the definition of rho