Merge pull request idaholab#29597 from miaoyinb/combinerFix

Fix the Positioning Bug of CombinerGenerator
farscape-project · Dec 22, 2024 · 90a2914 · 90a2914
2 parents 5b1df2e + 842fa6b
commit 90a2914
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 2 deletions.
diff --git a/framework/doc/content/media/meshgenerators/combiner_multi_translate.png b/framework/doc/content/media/meshgenerators/combiner_multi_translate.png
diff --git a/framework/doc/content/source/meshgenerators/CombinerGenerator.md b/framework/doc/content/source/meshgenerators/CombinerGenerator.md
@@ -40,7 +40,7 @@ If you specify `positions` then the number of `positions` must match the number
 
 Will generate a mesh that looks like:
 
-!media media/meshgenerators/combiner_multi_translate.png style=width:75%;
+!media framework/meshgenerators/combiner_multi_translate.png style=width:75%;
 
 Alternatively, the same displacement vectors can be supplied in a file with the `positions_file` option. The above mesh can equivalently be generated with the following.
 

diff --git a/framework/src/meshgenerators/CombinerGenerator.C b/framework/src/meshgenerators/CombinerGenerator.C
@@ -144,6 +144,13 @@ CombinerGenerator::generate()
     if (!mesh)
       paramError("inputs", _input_names[0], " is not a valid unstructured mesh");
 
+    // Move the first input mesh if applicable
+    if (_positions.size())
+    {
+      MeshTools::Modification::translate(
+          *mesh, _positions[0](0), _positions[0](1), _positions[0](2));
+    }
+
     // Read in all of the other meshes
     for (MooseIndex(_meshes) i = 1; i < _meshes.size(); ++i)
     {

diff --git a/large_media b/large_media
diff --git a/...ests/meshgenerators/combiner_generator/gold/combiner_multi_input_translate_from_file_in.e b/...ests/meshgenerators/combiner_generator/gold/combiner_multi_input_translate_from_file_in.e
diff --git a/test/tests/meshgenerators/combiner_generator/gold/combiner_multi_input_translate_in.e b/test/tests/meshgenerators/combiner_generator/gold/combiner_multi_input_translate_in.e