ClimaCoupler.jl icon indicating copy to clipboard operation
ClimaCoupler.jl copied to clipboard

Small performance improvements

Open juliasloan25 opened this issue 9 months ago • 0 comments

Minor issues found in https://github.com/CliMA/ClimaCoupler.jl/issues/390 that we want to clean up for better performance.

### Tasks
- [ ] don't pass FT as an argument
- [ ] wrap `pi` in FT for calculations
- [ ] use `zero(FT)` instead of `FT(0)` to reduce allocations
- [ ] remove use of `parent` (except where needed when spaces aren't identical ?)
- [ ] make sure no closures over global variables (e.g. `FT`
- [ ] limit use of `Val` to only when necessary

some points taken from https://github.com/CliMA/ClimaCoupler.jl/issues/323

juliasloan25 avatar May 03 '24 00:05 juliasloan25