Closed glwagner closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 90.93%. Comparing base (
46f51af
) to head (808b6b6
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@navidcy @charleskawczynski @simone-silvestri this passes now.
Can we please rebase and squash this PR? I like having a tidy and linear history.
Why?
Why?
It's good practice. I can do it. Just let me know when you're ready to merge, @glwagner.
I disagree that it's good practice for us. The PR is ready.
I added the examples in the Docs via Literate. Feel free to revert if you don't think this is a good idea.
This PR re-opens #172.
This PR tweaks the internal API for defining parameter sets to support user-defined parameters. This is important, because it will allow users to invoke functions in
Thermodynamics.jl
without requiring the full suite of parameters inThermodynamicsParameters
. For example, if one is only interested in computing density as a function of temperature, pressure, and (total) specific humidity, then they only require the universal gas constant and the molar masses of air and water.Next, this PR adds an example which illustrates such a parameter set, called
ConstitutiveParameters
. It culminates in this plot:which uses sea level data derived from the JRA55 reanalysis (subsampled to make the dataset smaller), showing the primary dependence of density on temperature, secondary dependence on pressure, and teritiary dependence on specific humidity for the typical conditions of the Earth's atmosphere at sea level.
Currently, there are no examples, so this PR introduces the first one. I'd be happy to turn the example into something that appears in the documentation via Literate (the example is very close to that form already).