Ignore false-positive use-before-assign warnings from pylint

illinois-ceesd · May 21, 2024 · 2d23628 · 2d23628
1 parent 279d40b
commit 2d23628
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/meshmode/discretization/connection/face.py b/meshmode/discretization/connection/face.py
@@ -311,7 +311,7 @@ def make_face_restriction(actx, discr, group_factory, boundary_tag,
 
             face_unit_nodes = face.map_to_volume(bdry_unit_nodes)
             resampling_mat = mp.resampling_matrix(
-                    vol_basis,
+                    vol_basis,  # pylint: disable=possibly-used-before-assignment
                     face_unit_nodes, mgrp.unit_nodes)
 
             # }}}
@@ -321,11 +321,11 @@ def make_face_restriction(actx, discr, group_factory, boundary_tag,
             # Find vertex_indices
             glob_face_vertices = mgrp.vertex_indices[
                     batch_boundary_el_numbers_in_grp][:, face.volume_vertex_indices]
-            vertex_indices[new_el_numbers] = \
-                    vol_to_bdry_vertices[glob_face_vertices]
+            vertex_indices[new_el_numbers] = (  # pylint: disable=possibly-used-before-assignment  # noqa: E501
+                    vol_to_bdry_vertices[glob_face_vertices])
 
             # Find nodes
-            nodes[:, new_el_numbers, :] = np.einsum(
+            nodes[:, new_el_numbers, :] = np.einsum(  # pylint: disable=possibly-used-before-assignment  # noqa: E501
                     "ij,dej->dei",
                     resampling_mat,
                     mgrp.nodes[:, batch_boundary_el_numbers_in_grp, :])

diff --git a/test/test_mesh.py b/test/test_mesh.py
@@ -908,6 +908,9 @@ def test_box_boundary_tags(dim, nelem, mesh_type, group_cls, visualize=False):
         nelements_per_axis = (nelem,)*3
         btag_to_face = {"btag_test_1": ["+x", "-y", "-z"],
                         "btag_test_2": ["+y", "-x", "+z"]}
+    else:
+        raise AssertionError("unexpected dim")
+
     mesh = mgen.generate_regular_rect_mesh(a=a, b=b,
                                       nelements_per_axis=nelements_per_axis, order=3,
                                       boundary_tag_to_face=btag_to_face,

diff --git a/test/test_partition.py b/test/test_partition.py
@@ -314,8 +314,8 @@ def test_partition_mesh(mesh_size, num_parts, num_groups, dim, scramble_parts):
                     assert found_reverse_adj, ("InterPartAdjacencyGroup is not "
                         "consistent")
 
-                    p_grp_num = find_group_indices(mesh.groups, p_meshwide_elem)
-                    p_n_grp_num = find_group_indices(mesh.groups, p_meshwide_n_elem)
+                    p_grp_num = find_group_indices(mesh.groups, p_meshwide_elem)  # pylint: disable=possibly-used-before-assignment  # noqa: E501
+                    p_n_grp_num = find_group_indices(mesh.groups, p_meshwide_n_elem)  # pylint: disable=possibly-used-before-assignment  # noqa: E501
 
                     p_elem_base = mesh.base_element_nrs[p_grp_num]
                     p_n_elem_base = mesh.base_element_nrs[p_n_grp_num]