ruff issues on test done

Author: hirish99

.gitignore

@@ -23,4 +23,5 @@ sumpy/_git_rev.py
 
 *.vts
 test/hirish_plotting
+test/test_eigenvalues.ipynb
 .vscode/settings.json

sumpy/recurrence_qbx.py

@@ -34,7 +34,7 @@
 """
 
 import math
-from collections.abc import Sequence
+from typing import TYPE_CHECKING
 
 import numpy as np
 import sympy as sp
@@ -47,6 +47,10 @@
 )
 
 
+if TYPE_CHECKING:
+    from collections.abc import Sequence
+
+
 # ================ Transform/Rotate =================
 def _produce_orthogonal_basis(normals: np.ndarray) -> Sequence[np.ndarray]:
     ndim, ncenters = normals.shape

test/test_recurrence.py

@@ -96,7 +96,7 @@ def _qbx_lp_general(knl, sources, targets, centers, radius,
     return result_qbx
 
 
-def _create_ellipse(n_p, mode_nr = 10, quad_convg_rate=100, a=2):
+def _create_ellipse(n_p, mode_nr=10, quad_convg_rate=100, a=2):
     t = np.linspace(0, 2 * np.pi, n_p, endpoint=False)
 
     phi = sp.symbols("phi")
@@ -322,24 +322,39 @@ def _construct_laplace_axis_2d(orders, resolutions):
 def plot():
     import matplotlib.pyplot as plt
     orders = [5, 7, 9, 11]
-    colors = ['b', 'g', 'r', 'c']
+    colors = ["b", "g", "r", "c"]
     resolutions = [2000, 300, 4000]
     err_mat, err_mat1 = _construct_laplace_axis_2d(orders, resolutions)
 
-    fig, ax1 = plt.subplots(1, 1, sharey=True, figsize=(6, 6))
+    _fig, ax1 = plt.subplots(1, 1, sharey=True, figsize=(6, 6))
 
     ax1.set_yscale("log")
     for i in range(len(orders)):
-        ax1.scatter(9.68845/np.array(resolutions), np.array(err_mat[i]), marker='+', label="$u = u_{qbxrec}$ ("+"$p_{QBX}$="+str(orders[i])+ ")", c=colors[i], s=50)
-        ax1.scatter(9.68845/np.array(resolutions), np.array(err_mat1[i]), marker='x', label="$u = u_{qbx}$ ("+"$p_{QBX}$="+str(orders[i]) + ")", c=colors[i], s=50)
+        ax1.scatter(
+            9.68845/np.array(resolutions), np.array(err_mat[i]),
+            marker="+",
+            label="$u = u_{qbxrec}$ ("
+            + "$p_{QBX}$=" + str(orders[i]) + ")",
+            c=colors[i], s=50)
+        ax1.scatter(
+            9.68845/np.array(resolutions), np.array(err_mat1[i]),
+            marker="x",
+            label="$u = u_{qbx}$ ("
+            + "$p_{QBX}$=" + str(orders[i]) + ")",
+            c=colors[i], s=50)
 
     ax1.set_xlabel("Mesh Resolution ($h$)", fontsize=14)
-    ax1.set_ylabel("Relative Error ($L_{\infty}$)", fontsize=14)
+    ax1.set_ylabel(r"Relative Error ($L_{\infty}$)", fontsize=14)
     ax1.set_title("$(u-u_{true})/u_{true}$", fontsize=16)
     ax1.legend()
 
-    plt.suptitle("Laplace 2D: Ellipse SLP Boundary Evaluation Error ($m=100$, $p_{offaxis}=8$)", fontsize=16)
-    plt.savefig("../../S_on_surface_convergence.pgf", bbox_inches='tight', pad_inches=0)
+    plt.suptitle(
+        "Laplace 2D: Ellipse SLP Boundary Evaluation Error"
+        " ($m=100$, $p_{offaxis}=8$)", fontsize=16)
+    plt.savefig(
+        "../../S_on_surface_convergence.pgf",
+        bbox_inches="tight", pad_inches=0)
     plt.show()
 
-plot()
+
+plot()

test/test_recurrence_qbx.py

@@ -96,7 +96,7 @@ def _qbx_lp_general(knl, sources, targets, centers, radius,
     return result_qbx
 
 
-def _create_ellipse(n_p, mode_nr = 10, quad_convg_rate=100, a=2):
+def _create_ellipse(n_p, mode_nr=10, quad_convg_rate=100, a=2):
     t = np.linspace(0, 2 * np.pi, n_p, endpoint=False)
 
     phi = sp.symbols("phi")
@@ -322,24 +322,39 @@ def _construct_laplace_axis_2d(orders, resolutions):
 def plot():
     import matplotlib.pyplot as plt
     orders = [5, 7, 9, 11]
-    colors = ['b', 'g', 'r', 'c']
+    colors = ["b", "g", "r", "c"]
     resolutions = [2000, 300, 4000]
     err_mat, err_mat1 = _construct_laplace_axis_2d(orders, resolutions)
 
-    fig, ax1 = plt.subplots(1, 1, sharey=True, figsize=(6, 6))
+    _fig, ax1 = plt.subplots(1, 1, sharey=True, figsize=(6, 6))
 
     ax1.set_yscale("log")
     for i in range(len(orders)):
-        ax1.scatter(9.68845/np.array(resolutions), np.array(err_mat[i]), marker='+', label="$u = u_{qbxrec}$ ("+"$p_{QBX}$="+str(orders[i])+ ")", c=colors[i], s=50)
-        ax1.scatter(9.68845/np.array(resolutions), np.array(err_mat1[i]), marker='x', label="$u = u_{qbx}$ ("+"$p_{QBX}$="+str(orders[i]) + ")", c=colors[i], s=50)
+        ax1.scatter(
+            9.68845/np.array(resolutions), np.array(err_mat[i]),
+            marker="+",
+            label="$u = u_{qbxrec}$ ("
+            + "$p_{QBX}$=" + str(orders[i]) + ")",
+            c=colors[i], s=50)
+        ax1.scatter(
+            9.68845/np.array(resolutions), np.array(err_mat1[i]),
+            marker="x",
+            label="$u = u_{qbx}$ ("
+            + "$p_{QBX}$=" + str(orders[i]) + ")",
+            c=colors[i], s=50)
 
     ax1.set_xlabel("Mesh Resolution ($h$)", fontsize=14)
-    ax1.set_ylabel("Relative Error ($L_{\infty}$)", fontsize=14)
+    ax1.set_ylabel(r"Relative Error ($L_{\infty}$)", fontsize=14)
     ax1.set_title("$(u-u_{true})/u_{true}$", fontsize=16)
     ax1.legend()
 
-    plt.suptitle("Laplace 2D: Ellipse SLP Boundary Evaluation Error ($m=100$, $p_{offaxis}=8$)", fontsize=16)
-    plt.savefig("../../S_on_surface_convergence.pgf", bbox_inches='tight', pad_inches=0)
+    plt.suptitle(
+        "Laplace 2D: Ellipse SLP Boundary Evaluation Error"
+        " ($m=100$, $p_{offaxis}=8$)", fontsize=16)
+    plt.savefig(
+        "../../S_on_surface_convergence.pgf",
+        bbox_inches="tight", pad_inches=0)
     plt.show()
 
-plot()
+
+plot()