-
Notifications
You must be signed in to change notification settings - Fork 235
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
[StructuralMechanicsApplication] Using AtomicAdd
in BaseSolidElement
#12180
base: master
Are you sure you want to change the base?
Conversation
Ups @AlejandroCornejo I wanted to modief the structural elements and I ended modiying yours :P |
// KRATOS_CHECK_VARIABLE_IN_NODAL_DATA(VELOCITY,r_node) | ||
// KRATOS_CHECK_VARIABLE_IN_NODAL_DATA(ACCELERATION,r_node) | ||
|
||
KRATOS_CHECK_DOF_IN_NODE(DISPLACEMENT_X, r_node) |
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.
is it better like this?
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.
It is the notation
* @return The Rayleigh alpha parameter. | ||
*/ | ||
double GetRayleighAlpha( | ||
const Properties& rProperties, |
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.
did you intend to change the elements from the FemToDemApp
?
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.
By error :(
📝 Description
Using
AtomicAdd
inBaseSolidElement
. Also adding some missing documentation and some minor clean up.🆕 Changelog
AtomicAdd
inBaseSolidElement