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

Updates to concrete moisture transport models #225

Merged
merged 3 commits into from
May 13, 2021

Conversation

bwspenc
Copy link
Collaborator

@bwspenc bwspenc commented May 11, 2021

ref #216 ref #223 ref #224

This set of changes resolves several issues with the models for moisture transport in concrete:

  1. There was inconsistency in the variable that some of the kernels for terms in the PDE were operating on
  2. We did not have active testing for this capability
  3. There was a bug in a BC used in these models

@moosebuild
Copy link

Job Precheck on e109f2e wanted to post the following:

Your code requires style changes.

A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:

curl -s https://mooseframework.inl.gov/blackbear/docs/PRs/225/style.patch | git apply -v

Alternatively, with your repository up to date and in the top level of your repository:

git clang-format ed737ef5e455b8e9cb108ee66884eabbbc2b5630

@moosebuild
Copy link

moosebuild commented May 11, 2021

Job Test on 7bad11a wanted to post the following:

View the site here

This comment will be updated on new commits.

@bwspenc
Copy link
Collaborator Author

bwspenc commented May 12, 2021

Based on discussions with @SudiptaBiswas and some new papers we have looked at, I'm not making the changes to the PDE terms that I had in the original version of this PR. We still need to have further discussions on this topic.

@@ -66,8 +66,6 @@ ConcreteThermalMoisture::validParams()
params.addParam<Real>("aggregate_mass", 1877.0, "aggregate mass (kg) per m^3");
params.addParam<Real>("water_to_cement_ratio", 0.43, "water to cement ratio");
params.addParam<Real>("concrete_cure_time", 23.0, "concrete curing time in days");
params.addDeprecatedParam<Real>(
"aggregate_vol_fraction", 0.7, "volumetric fraction of aggregates", "Not used");
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is failing the ASR assessment cases.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I'll remove that commit from this PR. It's because there hasn't been a successful update of the blackbear submodule in grizzly for a while. I almost have that fixed.

bwspenc added 3 commits May 12, 2021 21:03
…lab#223

This BC has an equation that computes the relative humidity to be
prescribed using a Dirichlet BC from a formula that is dependent on the
temperature, which should be converted from C to K.  However, the code
only did that conversion in one of the places where the temperature was
used. This corrects that error.
1) Delete the existing 2D test
2) Add a coarse, fast-running 1D test of the MAQBETH experiment. This
   runs with larger time steps and a coarser mesh than we would prefer,
   but the results are reasonably close to a more refined version, and
   reasonably close to the published experimental results.
3) Add a Python script for plotting the results of this test at various
   times reported in the paper on this experiment.
Copy link
Collaborator

@SudiptaBiswas SudiptaBiswas left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@SudiptaBiswas SudiptaBiswas merged commit ef3f3a6 into idaholab:devel May 13, 2021
@bwspenc bwspenc deleted the test_conc branch May 17, 2021 20:46
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.

3 participants