Skip to content
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

[GeoMechanicsApplication] Add cross-sectional area to the thermal line element #12349

Merged

Conversation

mnabideltares
Copy link
Contributor

📝 Description
The line element has to be extended to include the sectional area. All terms in the heat equation has to be multiplied with the sectional area. In principle, it does not make change in the results for an isolated 1D line element. The effect will appear when the 1D line element gets combined with 2D element, for example, when a well or pipe inside the soil gets simulated. The matrices of this 1D element will be superposined with the 2D elements in the main matrix.

🆕 Changelog

  • Added cross-section area to the thermal line element
  • Added "CROSS_AREA" to MaterialParameter.json
  • Removed duplicated files of MaterialParameter.json by putting them in a common dairectory

Copy link
Contributor

@WPK4FEM WPK4FEM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Mohammed,
Thank you, this removes a good amount of duplication by reuse of the MaterialProperties.
In this way all existing tests keep giving the same answer. It would be good if a future test that combines line and volume (or plane) elements contains a CROSS_AREA that is not 1.0

Copy link
Contributor

@rfaasse rfaasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! If this same issue is there for the Pw line elements, would you like to do that as part of this PR, or a separate one? Leaving that up to you, looks good to go from my side

@mnabideltares mnabideltares merged commit ef319a6 into master May 6, 2024
11 checks passed
@mnabideltares mnabideltares deleted the geo/12346-add-sectional-area-thermal-line-element branch May 6, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[GeoMechanicsApplication] Add cross-sectional area to the thermal line element
3 participants