From 792c84298dd27606549e8e37e84d8ab318a7ba72 Mon Sep 17 00:00:00 2001 From: Andreas Kloeckner Date: Tue, 21 May 2024 13:09:30 -0500 Subject: [PATCH] Ignore false-positive use-before-assign warnings from pylint --- meshmode/discretization/connection/face.py | 8 ++++---- test/test_mesh.py | 3 +++ test/test_partition.py | 4 ++-- 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/meshmode/discretization/connection/face.py b/meshmode/discretization/connection/face.py index 4c6b6b01..875c3d8d 100644 --- 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 index 22cd615e..23172b6e 100644 --- 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 index d17b5ac1..9a66661d 100644 --- 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]