Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enthalpy correction fix #68

Merged
merged 4 commits into from
Jul 28, 2016
Merged

Conversation

nagelt
Copy link
Member

@nagelt nagelt commented Jul 28, 2016

This PR moves a hard coded magic number to input file control. It also fixes some T2HC benchmarks.

@nagelt nagelt added WIP and removed WIP labels Jul 28, 2016
@@ -689,6 +689,8 @@ double CFiniteElementStd::CalCoef_RHS_TES(const int dof_index)
// AB + \Delta H <--> A + B
H_vap = - SolidProp->reaction_enthalpy;
//enthalpy correction
if (SolidProp->Capacity_mode == 4 || SolidProp->Capacity_mode == 5)
{
const double rhoSR = gp_ele->rho_s_curr[gp];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix indentation from this line

@nagelt nagelt force-pushed the enthalpy_correction_fix branch from 2329814 to 9422a0c Compare July 28, 2016 13:35
@nagelt nagelt force-pushed the enthalpy_correction_fix branch from 9422a0c to 28d87ec Compare July 28, 2016 13:46
@norihiro-w norihiro-w merged commit b5cb656 into ufz:master Jul 28, 2016
@norihiro-w
Copy link
Contributor

@nagelt thanks! this PR fixed all T2HC and TES benchmarks

@nagelt nagelt deleted the enthalpy_correction_fix branch August 12, 2016 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants