Performance roadmap

[Sbozzolo]

Tasks

Beta Give feedback

1. top_center_to_surface allocates at every step - see Reduce allocations and resolution dependence in how we get surface values from a center field #615
2. construct_atmos_ts allocates at every step #588
   +0
   
3. Remove dss! if not needed
4. Move to ClimaDiagnositcs to avoid the saving callback
5. Address type instabilties in ClimaUtilities
6. Preallocate space for regridded fields, add regridded_snapshot! to ClimaUtilities
7. Check norm2
8. Boundary fluxes
9. update_aux in Biogeochem

Attached is a flamegraph for a GPU run for a bucket with resolution 100. This comes from the run introduced in https://github.com/CliMA/ClimaLand.jl/pull/577/files#diff-c59e565d751c44123440feac3f10d5aaf78e58080716fafa54176cdbe44b50cd, with a simulation length of 1 day.

[juliasloan25]

the top_center_to_surface allocations will be reduced in #542 - see changes. I'm happy to move to a separate PR if preferred

[Sbozzolo]

the top_center_to_surface allocations will be reduced in #542 - see changes. I'm happy to move to a separate PR if preferred

Reduced or removed?

[juliasloan25]

We've implemented enough performance improvements for our current needs, so we can close this issue