Skip to content

New line geom mesh relation#972

Merged
jginternational merged 5 commits intomasterfrom
new-line-geom-mesh-relation
May 30, 2024
Merged

New line geom mesh relation#972
jginternational merged 5 commits intomasterfrom
new-line-geom-mesh-relation

Conversation

@jginternational
Copy link
Copy Markdown
Member

@jginternational jginternational commented May 29, 2024

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 :
image

  • 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.

@KratosMultiphysics/structural-mechanics

@AlejandroCornejo by petition of @RiccardoRossi

@jginternational jginternational merged commit 3f4cbda into master May 30, 2024
@jginternational jginternational deleted the new-line-geom-mesh-relation branch May 30, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant