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.

Sign up for GitHub

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

Jump to bottom

Add additional parameter for watertight nodeset and sideset check #29077

Open
DanielYankura opened this issue Nov 14, 2024 · 0 comments · May be fixed by #29136
Open

Add additional parameter for watertight nodeset and sideset check #29077

DanielYankura opened this issue Nov 14, 2024 · 0 comments · May be fixed by #29136
Labels
T: task An enhancement to the software.

Comments

@DanielYankura
Copy link
Contributor

Motivation

There are times when users may want a single sideset/nodeset to cover an entire mesh. Adding a parameter would allow users to specify which sidesets should be checked for.

Linked to #25278, #28816, and #28822

Design

This parameter should accept one or more sideset ids. If specified, instead of checking if an external node/side has a corresponding nodeset/sideset, it should check if it's associate with the ones provided by the parameter.

Impact

This will add additional capabilities to the mesh diagnostic system and give users more tools for debugging their meshes.
@DanielYankura DanielYankura added the T: task An enhancement to the software. label Nov 14, 2024
DanielYankura added a commit to DanielYankura/moose that referenced this issue Nov 25, 2024
@DanielYankura
Added parameters for watertight sideset and nodeset diagnostic check
3c505cd 
-Diagnostics can now take an additional vector parameter _watertight_boundaries
-Users can use it to input a list of sidesets/nodesets
-If supplied the diagnostic check functionality changes
-It will instead check if an external side/node is assigned to one of the provided boundary ids
-The error message for these instances is worded differently to reflect what the diagnostic check is looking for
-The t in checkWaterTightSidesets is now lowercase because it was bothering me and to match checkWatertightNodesets

closes idaholab#29077
@DanielYankura DanielYankura linked a pull request Nov 25, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: task An enhancement to the software.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added parameters for watertight sideset and nodeset diagnostic check DanielYankura/moose
1 participant
@DanielYankura