cb-geo / mpm

CB-Geo High-Performance Material Point Method
https://www.cb-geo.com/research/mpm
Other
235 stars 82 forks source link

Feature/state vars smoothing #699

Closed jgiven100 closed 2 years ago

jgiven100 commented 3 years ago

Describe the PR

Related Issues/PRs https://github.com/cb-geo/mpm/issues/694

Additional context The proposed implementation requires the following additional information in the input .json:

  "analysis": {
    ...,
    "state_variable_smoothing":[
      {
        "smoothing" : true,
        "state_variable" : "pdstrain"
      }
    ]
  },
codecov[bot] commented 3 years ago

Codecov Report

Merging #699 (073fac4) into develop (70da0c5) will decrease coverage by 0.16%. The diff coverage is 6.52%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #699      +/-   ##
===========================================
- Coverage    96.77%   96.61%   -0.16%     
===========================================
  Files          130      130              
  Lines        25834    25880      +46     
===========================================
+ Hits         24999    25002       +3     
- Misses         835      878      +43     
Impacted Files Coverage Δ
include/node.h 93.55% <0.00%> (-6.45%) :arrow_down:
include/node_base.h 100.00% <ø> (ø)
include/particles/particle.h 100.00% <ø> (ø)
include/particles/particle.tcc 92.21% <0.00%> (-3.00%) :arrow_down:
include/particles/particle_base.h 100.00% <ø> (ø)
include/solvers/mpm_scheme/mpm_scheme.tcc 63.10% <0.00%> (-4.85%) :arrow_down:
include/node.tcc 92.56% <7.69%> (-3.42%) :arrow_down:
include/solvers/mpm_explicit.tcc 89.71% <33.33%> (-5.46%) :arrow_down:

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 70da0c5...073fac4. Read the comment docs.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jgiven100 commented 2 years ago

Not considering this approach anymore -> closing.