Closed ischoegl closed 3 months ago
Attention: Patch coverage is 59.52381%
with 17 lines
in your changes missing coverage. Please review.
Project coverage is 73.13%. Comparing base (
9d0e54a
) to head (682bde8
). Report is 31 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
src/clib/ct.cpp | 59.52% | 17 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Changes proposed in this pull request
Expose partial molar thermo properties to MATLAB, and remove mislabeled reference state
enthalpies_RT
. The fix sidesteps the discussion of "standard state" vs "reference state" nomenclature of #522, and instead focuses on quantities that are relevant to the actual state. Added properties are:ThermoPhase.partialMolarEnthalpies
ThermoPhase.partialMolarEntropies
ThermoPhase.partialMolarIntEnergies
ThermoPhase.partialMolarCp
ThermoPhase.partialMolarVolumes
ThermoPhase.electrochemicalPotentials
If applicable, fill in the issue number this pull request is fixing
Closes #522
If applicable, provide an example illustrating new features this pull request is introducing
Checklist
scons build
&scons test
) and unit tests address code coverage