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

Cdpm2rate #61

Merged
merged 2 commits into from
Nov 20, 2019
Merged

Cdpm2rate #61

merged 2 commits into from
Nov 20, 2019

Conversation

githubgrasp
Copy link
Contributor

Added more control options for modelling rate dependence in CDPM2.
Added tests for these features
Added additional parameters to doc and fixed some problems with the CDPM2 description

@codecov
Copy link

codecov bot commented Nov 17, 2019

Codecov Report

Merging #61 into master will increase coverage by <.01%.
The diff coverage is 88.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   39.82%   39.83%   +<.01%     
==========================================
  Files        1241     1241              
  Lines      125579   125596      +17     
==========================================
+ Hits        50014    50026      +12     
- Misses      75565    75570       +5
Impacted Files Coverage Δ
src/sm/Materials/ConcreteMaterials/concretedpm2.h 93.33% <ø> (ø) ⬆️
src/sm/Materials/ConcreteMaterials/concretedpm2.C 81.71% <88.46%> (-0.13%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7071459...ab35723. Read the comment docs.

@githubgrasp
Copy link
Contributor Author

Is everything OK with this pull request? Can someone merge it?

@bpatzak bpatzak merged commit 363bb26 into oofem:master Nov 20, 2019
@githubgrasp githubgrasp deleted the cdpm2rate branch January 16, 2021 09:12
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.

None yet

2 participants