Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #296 +/- ##
==========================================
- Coverage 87.00% 86.29% -0.72%
==========================================
Files 22 22
Lines 1093 1102 +9
==========================================
Hits 951 951
- Misses 142 151 +9 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
szy21
left a comment
There was a problem hiding this comment.
Thanks. I'm fine with not maintaining the backward compatibility though. I plan to update ClimaAtmos with the new Thermodynamics as soon as SurfaceFluxes is done.
I thought this may allow us to work more piecemeal in the interim, working in atmos, coupler, land in sequence. I do want to delete these wrappers and other redundant code as soon as we can though. |
Adds wrappers with old function names for backward compatibility of ClimaAtmos, ClimaLand etc. with this version of Thermodynamics.jl
This PR may allow us to release this version of TD and then make the downstream changes, before removing the deprecated code.