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

Add RGMB option to enable flexible assembly stitching #28231

Merged
merged 9 commits into from
Aug 4, 2024

Conversation

shikhar413
Copy link
Contributor

Reason

Design

Impact

refs #28230

@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch 2 times, most recently from dc30c26 to f4ffff8 Compare July 25, 2024 20:01
@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from f4ffff8 to 134cb93 Compare July 25, 2024 20:23
Copy link
Contributor

@miaoyinb miaoyinb left a 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

@moosebuild
Copy link
Contributor

moosebuild commented Jul 25, 2024

Job Documentation on 3e3e0c9 wanted to post the following:

View the site here

This comment will be updated on new commits.

Copy link
Contributor

@miaoyinb miaoyinb left a 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.

@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from 134cb93 to 1fe197c Compare July 29, 2024 13:16
@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from 7036d88 to 76527dd Compare July 29, 2024 14:08
@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from 324cf9f to c2e2667 Compare July 29, 2024 15:03
@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from 597e0ca to 0b3da71 Compare July 29, 2024 17:13
@shikhar413
Copy link
Contributor Author

@GiudGiud @miaoyinb all comments should be addressed now, will push another commit to address remaining test failure in a bit

Copy link
Contributor

@miaoyinb miaoyinb left a 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.

@moosebuild
Copy link
Contributor

moosebuild commented Jul 30, 2024

Job Coverage on 3e3e0c9 wanted to post the following:

Framework coverage

Coverage did not change

Modules coverage

Reactor

468ab7 #28231 3e3e0c
Total Total +/- New
Rate 93.70% 93.61% -0.09% 91.28%
Hits 7304 7430 +126 199
Misses 491 507 +16 19

Diff coverage report

Full coverage report

Full coverage reports

Reports

This comment will be updated on new commits.

@GiudGiud GiudGiud assigned GiudGiud and unassigned miaoyinb Jul 31, 2024
@GiudGiud
Copy link
Contributor

I'll review this when I get back unless someone else does it first
@miaoyinb thank you for reviewing!

@shikhar413 shikhar413 force-pushed the rgmb_flexible_assembly_stitching branch from 6122d1d to 3e3e0c9 Compare August 4, 2024 16:37
@moosebuild
Copy link
Contributor

Job Precheck on 6122d1d wanted to post the following:

Your code requires style changes.

A patch was auto generated and copied here
You can directly apply the patch by running, in the top level of your repository:

curl -s https://mooseframework.inl.gov/docs/PRs/28231/clang_format/style.patch | git apply -v

Alternatively, with your repository up to date and in the top level of your repository:

git clang-format 468ab73aa23733be91b7e142dd2926aeb230ff53

large_media Show resolved Hide resolved
@GiudGiud
Copy link
Contributor

GiudGiud commented Aug 4, 2024

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

@GiudGiud GiudGiud merged commit 6630425 into idaholab:next Aug 4, 2024
47 of 49 checks passed
@shikhar413 shikhar413 deleted the rgmb_flexible_assembly_stitching branch August 4, 2024 21:49
@milljm
Copy link
Member

milljm commented Aug 5, 2024

@shikhar413, this merge seems to be breaking Macintosh Intel targets only. Ref: https://civet.inl.gov/job/2367026/.

[1.392s]     FAIL reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex FAILED (ERRMSG) [reactor-opt]

Error:

reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: *** ERROR ***
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: Found point (x,y,z)=( 2.04665, 1.11022e-16,        0) on hole boundary but outside outer boundary
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: Stack frames: 15
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 0: 0   libmesh_opt.0.dylib                 0x000000011145dc1e libMesh::print_trace(std::__1::basic_ostream<char, std::__1::char_traits<char>>&) + 1070
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 1: 1   libmesh_opt.0.dylib                 0x000000011145b8b3 libMesh::MacroFunctions::report_error(char const*, int, char const*, char const*, std::__1::basic_ostream<char, std::__1::char_traits<char>>&) + 179
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 2: 2   libmesh_opt.0.dylib                 0x0000000111a0145b libMesh::TriangulatorInterface::verify_holes(libMesh::TriangulatorInterface::Hole const&) + 731
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 3: 3   libmesh_opt.0.dylib                 0x0000000111a00a19 libMesh::TriangulatorInterface::elems_to_segments() + 4969
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 4: 4   libmesh_opt.0.dylib                 0x00000001119e9bcb libMesh::Poly2TriTriangulator::triangulate() + 91
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 5: 5   libmoose-opt.0.dylib                0x000000010f413bde XYDelaunayGenerator::generate() + 2990
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 6: 6   libmoose-opt.0.dylib                0x000000010f4b1c80 MeshGenerator::generateInternal() + 160
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 7: 7   libmoose-opt.0.dylib                0x00000001102407a5 MeshGeneratorSystem::executeMeshGenerators() + 1509
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 8: 8   libmoose-opt.0.dylib                0x000000010fe52335 ActionWarehouse::executeActionsWithAction(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&) + 1381
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 9: 9   libmoose-opt.0.dylib                0x000000010fe51bd8 ActionWarehouse::executeAllActions() + 280
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 10: 10  libmoose-opt.0.dylib                0x00000001101d81ff MooseApp::runInputFile() + 143
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 11: 11  libmoose-opt.0.dylib                0x00000001101d4542 MooseApp::run() + 1250
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 12: 12  reactor-opt                         0x000000010c6bdc45 void Moose::main<ReactorTestApp>(int, char**) + 133
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 13: 13  reactor-opt                         0x000000010c6bdbb9 main + 9
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 14: 14  dyld                                0x00007ff8059cb366 start + 1942
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: [0] ../src/mesh/triangulator_interface.C, line 449, compiled Jul 30 2024 at 17:46:23
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: Stack frames: 7
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 0: 0   libmesh_opt.0.dylib                 0x000000011145dc1e libMesh::print_trace(std::__1::basic_ostream<char, std::__1::char_traits<char>>&) + 1070
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 1: 1   libmoose-opt.0.dylib                0x00000001101de321 moose::internal::mooseErrorRaw(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>) + 769
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 2: 2   libmoose-opt.0.dylib                0x000000010fa5f0e1 void mooseError<char const*>(char const*&&) + 273
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 3: 3   libmoose-opt.0.dylib                0x00000001101d4ab3 MooseApp::run() + 2643
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 4: 4   reactor-opt                         0x000000010c6bdc45 void Moose::main<ReactorTestApp>(int, char**) + 133
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 5: 5   reactor-opt                         0x000000010c6bdbb9 main + 9
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 6: 6   dyld                                0x00007ff8059cb366 start + 1942
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: Abort(1) on node 0 (rank 0 in comm 0): application called MPI_Abort(MPI_COMM_WORLD, 1) - process 0
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: 
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: ################################################################################
reactor/test:meshgenerators/core_mesh_generator.flexible_stitching_hex: Tester failed, reason: ERRMSG

Any ideas? Can we fix this or perhaps do we need to skip this test, when it encounters Mac Intel?

@shikhar413
Copy link
Contributor Author

Hmm I'm not sure. This is most likely an issue with XYDelaunayTriangulator but I don't really have a way to reproduce this error since I don't have a Intel Mac environment.

Any ideas on this @GiudGiud @miaoyinb?

@miaoyinb
Copy link
Contributor

miaoyinb commented Aug 5, 2024

Hmm I'm not sure. This is most likely an issue with XYDelaunayTriangulator but I don't really have a way to reproduce this error since I don't have a Intel Mac environment.

Any ideas on this @GiudGiud @miaoyinb?

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"

@shikhar413
Copy link
Contributor Author

Thanks @miaoyinb let me put in a patch for this

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.

5 participants