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

Nonlocal Scalar Damage Models #21786

Closed
maxnezdyur opened this issue Aug 5, 2022 · 3 comments · Fixed by #21787 or #21944
Closed

Nonlocal Scalar Damage Models #21786

maxnezdyur opened this issue Aug 5, 2022 · 3 comments · Fixed by #21787 or #21944
Assignees
Labels
T: task An enhancement to the software.

Comments

@maxnezdyur
Copy link
Contributor

Reason

MOOSE has local damage models but they can exhibit mesh sensitivity. Nonlocal damage models allow for less mesh sensitivity while enabling the use of the same models. Damage averaging approach would be an easy first step that wouldn't require any change to existing damage models.

Design

Create new damage model that would work with any existing damage model. It takes in a some form of averging user object and creates new damage_index for ComputeDamageStress.

Impact

Would allow every current scalar local damage model to be used as a nonlocal version of itself.

@maxnezdyur maxnezdyur added the T: task An enhancement to the software. label Aug 5, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 8, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 8, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 10, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 10, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 10, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 10, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 11, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 16, 2022
maxnezdyur added a commit to maxnezdyur/moose that referenced this issue Aug 18, 2022
@bwspenc
Copy link
Contributor

bwspenc commented Aug 19, 2022

We discovered that the initial version of this fails parallel tests for higher processor counts because we are not communicating data from processors that have elements that fall within the averaging radius, but are not the owning processors of the current element's neighbors. @dschwen has an idea on a pretty easy fix for this.

bwspenc added a commit to bwspenc/moose that referenced this issue Aug 19, 2022
As noted on that issue, there's a parallel issue that we will be
addressing.
@loganharbour
Copy link
Member

We need an open issue for this for SQA

@dschwen dschwen reopened this Aug 23, 2022
@dschwen
Copy link
Member

dschwen commented Aug 23, 2022

We need an open issue for this for SQA

There ya go :-)

dschwen added a commit to dschwen/moose that referenced this issue Aug 23, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 25, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 25, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 25, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 26, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 26, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 26, 2022
dschwen added a commit to dschwen/moose that referenced this issue Aug 29, 2022
lynnmunday pushed a commit to lynnmunday/moose that referenced this issue Sep 8, 2022
lynnmunday pushed a commit to lynnmunday/moose that referenced this issue Sep 8, 2022
lynnmunday pushed a commit to lynnmunday/moose that referenced this issue Sep 8, 2022
lynnmunday pushed a commit to lynnmunday/moose that referenced this issue Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: task An enhancement to the software.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants