-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Take strain_free_density as initial value for density calculation #29773
base: next
Are you sure you want to change the base?
Take strain_free_density as initial value for density calculation #29773
Conversation
Job Documentation, step Docs: sync website on 443539c wanted to post the following: View the site here This comment will be updated on new commits. |
Job Coverage, step Generate coverage on 443539c wanted to post the following: Framework coverageCoverage did not change Modules coverageMisc
Full coverage reportsReports
This comment will be updated on new commits. |
Ready for you to take a look @bwspenc thanks! |
@tophmatthews I think it's in |
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 good to me. My only comment is that to me, it seems like it would be more natural to name the property name and input parameter initial_density
rather thanstrain_free_density
. Since that's how you chose to name your member variable that points to the input parameter I think you might feel the same way.
Hm, I thought I removed a lot of "Density" at some point. Could just be a |
I guess it's up to the user to use whatever material model is most appropriate to define |
Yeah good suggestion. The bare minimum need for |
Interestingly, there is an old comment in
I didn't touch this at this time, but it does make a good point that unless you are using displacements, you don't need |
I dont see |
it's in |
Creates new
MaterialPropertyName
strain_free_density
that will replace the now deprecateddensity
param. This enables the use of a material property for density independent of strains (say for example, dependent on chemistry), which can be changed throughout a simulation. Added test forDensity
inmodules/misc/test/tests
as well sinceDensity
is in themisc
module, but is only tested incombined
(almost seems like density should be insolid_mechanics
but whatevs).ping maybe @bwspenc and @dschwen ?
Closes #29556