Skip to content

[IgaApplication] [FluidDynamicsApplication] pressure condition for IGA stokes fluids #13518

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

Merged
merged 10 commits into from
Jun 18, 2025

Conversation

NickNick9
Copy link
Contributor

📝 Description
Addition of the pressure condition, where we impose weakly the pressure and the normal stress (Neumann conditions).

$$ \left( \boldsymbol{\tau} - p \mathbb{I} \right) \cdot \mathbf{n} = \mathbf{t}_N \quad \text{on } \Gamma_N $$

The corresponding test has been added.

@NickNick9 NickNick9 requested a review from a team as a code owner June 12, 2025 10:38
@NickNick9 NickNick9 requested review from andrewgorgi June 16, 2025 11:03
Copy link
Contributor

@andrewgorgi andrewgorgi left a comment

Choose a reason for hiding this comment

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

Minor comments


const auto& r_geometry = GetGeometry();
const SizeType number_of_nodes = r_geometry.size();
const GeometryType::ShapeFunctionsGradientsType& DN_De = r_geometry.ShapeFunctionsLocalGradients(r_geometry.GetDefaultIntegrationMethod());
Copy link
Contributor

Choose a reason for hiding this comment

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

r_DN_De


KRATOS_ERROR_IF(mDim == 3) << "SupportPressureCondition is not implemented in 3D. Current dimension: " << mDim << std::endl;
// Integration
const GeometryType::IntegrationPointsArrayType& integration_points = r_geometry.IntegrationPoints();
Copy link
Contributor

Choose a reason for hiding this comment

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

reference "r+..."

/// Create with Id, pointer to geometry and pointer to property
Condition::Pointer Create(
IndexType NewId,
NodesArrayType const& ThisNodes,
Copy link
Contributor

Choose a reason for hiding this comment

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

reference nomenclature

andrewgorgi
andrewgorgi previously approved these changes Jun 18, 2025
Copy link
Contributor

@andrewgorgi andrewgorgi left a comment

Choose a reason for hiding this comment

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

Looking forward for the next PR

Copy link
Contributor

@andrewgorgi andrewgorgi left a comment

Choose a reason for hiding this comment

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

Nice work

@NickNick9 NickNick9 merged commit 525f636 into master Jun 18, 2025
11 checks passed
@NickNick9 NickNick9 deleted the PR_pressure_condition branch June 18, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants