This PR will remove an annoying warning when using lines in Structural mechanics app
The feature may affect Structural > Beams > Hinges.
Until this PR we were getting the mdpa data:
Begin ElementalData CONDENSED_DOF_LIST // Group: Hinge Auto1
25 [3] (0,1,2)
38 [3] (3,4,5)
End ElementalData
Being 25 and 38 the starting node and final node of the line with boundary condition > Hinge
The way we used to pick this numbers was:
Get the list of nodes of the line
Sort them numerically (not following the line normal, just sort the ids)
Get min and max
This works fine since GiD meshes this way but:
Mesh could be renumbered
This does not preserve the direction of the geometrical line.
To get the relation between the geometrical line and the nodes, we were using an old gid feature called condition over body, wich caused annoying messages when fails
News
With this PR :
We'll keep the line direction. You can see it in the image, there is a pink arrow showing the normal, and a blue arrow showing the direction.
We get rid of this old gid feature, and start using the new feature: GiD Meshes and Geometries are attached, so we can navigate from a geometry line to it's mesh line elements, and viceversa.
No more annoying messages.
No need to increment the Minimum GiD version, this should work with 16.1.4d
The output will be:
Begin ElementalData CONDENSED_DOF_LIST // Group: Hinge Auto1
38 [3] (0,1,2)
25 [3] (3,4,5)
End ElementalData
Wich is fine, following the line direction, the first node is 38, and the last is 25.
This PR will remove an annoying warning when using lines in Structural mechanics app
The feature may affect Structural > Beams > Hinges.
Until this PR we were getting the mdpa data:
Being 25 and 38 the starting node and final node of the line with boundary condition > Hinge
The way we used to pick this numbers was:
This works fine since GiD meshes this way but:
News
With this PR :
The output will be:
Wich is fine, following the line direction, the first node is 38, and the last is 25.
@KratosMultiphysics/structural-mechanics
@AlejandroCornejo by petition of @RiccardoRossi