Skip to content

Add sliding_interface_tolerance and stationary_enclosed_entity parameters#1625

Merged
shreyas-flex merged 7 commits intomainfrom
shreyas/params-related-to-sliding-interface-fusion
Dec 10, 2025
Merged

Add sliding_interface_tolerance and stationary_enclosed_entity parameters#1625
shreyas-flex merged 7 commits intomainfrom
shreyas/params-related-to-sliding-interface-fusion

Conversation

@shreyas-flex
Copy link
Contributor

  • Adds two important parameters related to sliding interface intersecting wall feature:
  1. sliding_interface_tolerance : used to decide whether a point/curve lies on the sliding interface.
  2. stationary_enclosed_entities : used to denote to solver that certain bodies within sliding interface are not rotating with respect to frame of reference of overall simulation

@shreyas-flex shreyas-flex marked this pull request as ready for review December 10, 2025 16:56
@benflexcompute
Copy link
Collaborator

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@shreyas-flex shreyas-flex enabled auto-merge (squash) December 10, 2025 22:15
@shreyas-flex shreyas-flex merged commit d4d4b19 into main Dec 10, 2025
20 checks passed
@shreyas-flex shreyas-flex deleted the shreyas/params-related-to-sliding-interface-fusion branch December 10, 2025 22:19
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