-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add RGMB option to enable flexible assembly stitching #28231
Add RGMB option to enable flexible assembly stitching #28231
Conversation
dc30c26
to
f4ffff8
Compare
f4ffff8
to
134cb93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only reviewed FlexiblePattern part. May take a look at other parts later today. Thanks
Job Documentation on 3e3e0c9 wanted to post the following: View the site here This comment will be updated on new commits. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor comments. Thanks.
modules/reactor/test/tests/meshgenerators/core_mesh_generator/tests
Outdated
Show resolved
Hide resolved
modules/reactor/doc/content/source/meshgenerators/CoreMeshGenerator.md
Outdated
Show resolved
Hide resolved
modules/reactor/doc/content/source/meshgenerators/CoreMeshGenerator.md
Outdated
Show resolved
Hide resolved
modules/reactor/doc/content/source/meshgenerators/ReactorMeshParams.md
Outdated
Show resolved
Hide resolved
...est/tests/meshgenerators/core_mesh_generator/gold/core_flexible_assembly_stitching_cart_in.e
Outdated
Show resolved
Hide resolved
modules/reactor/doc/content/source/meshgenerators/FlexiblePatternGenerator.md
Outdated
Show resolved
Hide resolved
134cb93
to
1fe197c
Compare
7036d88
to
76527dd
Compare
324cf9f
to
c2e2667
Compare
597e0ca
to
0b3da71
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some final minor suggestions. Thanks.
Job Coverage on 3e3e0c9 wanted to post the following: Framework coverageCoverage did not change Modules coverageReactor
Full coverage reportsReports
This comment will be updated on new commits. |
I'll review this when I get back unless someone else does it first |
modules/reactor/doc/content/source/meshgenerators/CoreMeshGenerator.md
Outdated
Show resolved
Hide resolved
6122d1d
to
3e3e0c9
Compare
Job Precheck on 6122d1d wanted to post the following: Your code requires style changes. A patch was auto generated and copied here
Alternatively, with your repository up to date and in the top level of your repository:
|
Thanks for the PR @shikhar413 . I think this solves a big potential problem for a wide range of users, and thank you for reviewing @miaoyinb |
@shikhar413, this merge seems to be breaking Macintosh Intel targets only. Ref: https://civet.inl.gov/job/2367026/.
Error:
Any ideas? Can we fix this or perhaps do we need to skip this test, when it encounters Mac Intel? |
I think it might be due to "verify_holes=true" in FlexiblePatternGenerator. If so, it's a known issue libMesh/libmesh#3496. You can set that param as false to skip this check to avert this "false positive" |
Thanks @miaoyinb let me put in a patch for this |
Reason
Design
Impact
refs #28230