Closed longemen3000 closed 1 year ago
Merging #187 (7485dae) into master (86b8464) will decrease coverage by
0.45%
. The diff coverage is9.63%
.:exclamation: Current head 7485dae differs from pull request most recent head 2953bf0. Consider uploading reports for the commit 2953bf0 to get more accurate results
@@ Coverage Diff @@
## master #187 +/- ##
==========================================
- Coverage 85.70% 85.25% -0.45%
==========================================
Files 222 223 +1
Lines 15110 15154 +44
==========================================
- Hits 12950 12920 -30
- Misses 2160 2234 +74
Impacted Files | Coverage Δ | |
---|---|---|
ext/ClapeyronSymbolicsExt.jl | 0.00% <0.00%> (ø) |
|
src/Clapeyron.jl | 100.00% <ø> (ø) |
|
src/models/ideal/ideal.jl | 45.83% <ø> (-2.17%) |
:arrow_down: |
src/modules/solvers/Solvers.jl | 10.52% <ø> (ø) |
|
src/methods/VT.jl | 86.17% <100.00%> (ø) |
|
src/methods/differentials.jl | 100.00% <100.00%> (ø) |
|
src/models/Activity/equations.jl | 80.00% <100.00%> (ø) |
|
src/modules/solvers/ad.jl | 76.82% <100.00%> (+1.18%) |
:arrow_up: |
This PR add a Symbolics.jl extension to Clapeyron. we make a bunch of AD functions Symbolics-compatible. this allows us to pass Symbolic Variables into any (VT) bulk property and obtain a symbolic expression corresponding the evaluation of an specific eos at (V,T,z)::Symbolic conditions. one simple example:
Cp1 results in this expression:
This is, of course, the result of combinatorial explosion, a phenomena that affects calculation of Symbolic derivatives, but it is still the exact expression for the calculation of Cp(V,T) using Peng-Robinson.
Some things are not supported. notably:
For work on Symbolic EoS, i suppose that instead of solving single and multicomponent phase equilibria in the library, the system handling the generated expressions is the one in charge of solving the equilibrium conditions. This PR is a step on addressing #186.