-
Notifications
You must be signed in to change notification settings - Fork 17
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
Adding integrated material qois (dissipation for HyperVisco) #77
Conversation
…d an initial test
…l and free energies at a material point
…use it's being used to try some things out for now
…e functions" This reverts commit 0bd5216.
…ons: Fixed tests this time
…pdating test accordingly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. Thanks for putting it all together! I found one potential bug; see my comments. It's easy to fix.
optimism/material/MaterialModel.py
Outdated
'density','compute_material_qoi'], | ||
defaults=(0.0,)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a bug. The default values for namedtuple
are applied from the right, so I think this will now default the value for compute_material_qoi
to 0.0 instead of density
. I think you should put compute_material_qoi
before density
?
Don't merge just yet... i"m working on adding a uniaxial strain solution this morning |
… with a relaxation time of 25.0s and loading time of 100.0s at a strain rate of 1.0e-2. Removed the zero test since this is covered by this test now for time=0s. Will modify later to cover range of strain rates. This will lead to large test times though.
Here the total deformation graident we will write as The kinematics are given by The rate can be written for convenience as Now calculate Since this motion is rotation free, we can readily write with Need to assemble and solve This gives the following equation Set Choose an integrating factor as follows If loading stops at |
…red last and makes compute_material_qoi as default to None. Makes specification of MaterialModel functions explicit in implementations
@ralberd let me know when you're done with this guy. I'll give it a review and smash the merge button. |
I quickly added this, but perhaps there are better ways to get these quantities out.