[SMApp] Minor hotfixes in the Base solid element #13015
Conversation
AlejandroCornejo
added
the
FastPR
| @@ -747,6 +747,7 @@ void BaseSolidElement::CalculateOnIntegrationPoints( | |||
|
|
|||
| if ( rOutput.size() != number_of_integration_points ) | |||
| rOutput.resize( number_of_integration_points, false ); | |||
| rOutput.assign(number_of_integration_points, 0.0); | |||
There was a problem hiding this comment.
?, when you do resize does not assign the value you give?
There was a problem hiding this comment.
Otherwise maybe remove the false
There was a problem hiding this comment.
apparently not. Currently you get random numbers in the posts
There was a problem hiding this comment.
Okay I just checked, the resize only assigns value to the new elements
There was a problem hiding this comment.
Just for you to have an example. If I have 2 materials (elastic and damage) and I want to print DAMAGE, the elastic zone plots random values :S
There was a problem hiding this comment.
I would remove the false is misleading
There was a problem hiding this comment.
Just noting that false is to say that previous values are not kept when resizing but doesn't call the initializer of the stored type.
| @@ -747,6 +747,7 @@ void BaseSolidElement::CalculateOnIntegrationPoints( | |||
|
|
|||
| if ( rOutput.size() != number_of_integration_points ) | |||
| rOutput.resize( number_of_integration_points, false ); | |||
There was a problem hiding this comment.
| rOutput.resize( number_of_integration_points, false ); | |
| rOutput.resize( number_of_integration_points); |
Or 0.0 or simply remove it if we do assign later
There was a problem hiding this comment.
yes, the assing not only puts values but also resizes. I remove the resize
Basically defaulting to null the components of the output in the CalculateOnIntgerationPoints. Currently, if the CL does not return the value, the output are random values, not 0.