Don't send empty elem request vectors

roystgnr · roystgnr · commit b3870e7f7516 · 2023-04-17T12:56:41.000-05:00
I fixed this in the nodes case but didn't think to search for an
equivalent error elsewhere...

This should fix up our distributed-mesh-sweeps blocking our
devel-&gt;master merge.
diff --git a/src/mesh/exodusII_io.C b/src/mesh/exodusII_io.C
@@ -1150,9 +1150,12 @@ void ExodusII_IO::copy_elemental_solution(System & system,
       std::unordered_map<processor_id_type, std::vector<dof_id_type>> elem_ids_to_request;
       if (this->processor_id() != 0)
         {
-          std::vector<dof_id_type> & elem_ids = elem_ids_to_request[0];
+          std::vector<dof_id_type> elem_ids;
           for (auto & elem : mesh.active_local_element_ptr_range())
             elem_ids.push_back(elem->id());
+
+          if (!elem_ids.empty()
+            elem_ids_to_request[0] = std::move(elem_ids);
         }
 
       auto value_gather_functor =

Original file line number	Diff line number	Diff line change
`@@ -1150,9 +1150,12 @@ void ExodusII_IO::copy_elemental_solution(System & system,`
`1150`	`1150`	`std::unordered_map<processor_id_type, std::vector<dof_id_type>> elem_ids_to_request;`
`1151`	`1151`	`if (this->processor_id() != 0)`
`1152`	`1152`	`{`
`1153`		`- std::vector<dof_id_type> & elem_ids = elem_ids_to_request[0];`
	`1153`	`+ std::vector<dof_id_type> elem_ids;`
`1154`	`1154`	`for (auto & elem : mesh.active_local_element_ptr_range())`
`1155`	`1155`	`elem_ids.push_back(elem->id());`
	`1156`	`+`
	`1157`	`+ if (!elem_ids.empty()`
	`1158`	`+ elem_ids_to_request[0] = std::move(elem_ids);`
`1156`	`1159`	`}`
`1157`	`1160`
`1158`	`1161`	`auto value_gather_functor =`