From 81db4771e9b5a4e0e8826db05f7ee64540157783 Mon Sep 17 00:00:00 2001 From: Specter842 Date: Sat, 5 Sep 2026 06:19:12 +0530 Subject: [PATCH] enable two-center case in test_poisson_bvp_gives_the_correct_laplacian this was commented out as failing. it passes now with default params (verified locally, atol=1e-2, both on-grid and random points). --- src/grid/tests/test_poisson.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/grid/tests/test_poisson.py b/src/grid/tests/test_poisson.py index c8e9de749..2c32bbf93 100644 --- a/src/grid/tests/test_poisson.py +++ b/src/grid/tests/test_poisson.py @@ -236,11 +236,7 @@ def test_poisson_bvp_on_unit_charge_distribution(oned, tf, remove_large_pts, cen [zero_func, np.array([[0.0, 0.0, 0.0]])], [gauss, np.array([[0.0, 0.0, 0.0]])], [gauss, np.array([[1.0, 0.0, 0.0]])], - # TODO: Couldn't get the following test to pass. - # [ - # lambda pts, centers: gauss(pts, centers), - # np.array([[0.0, 0.0, 0.0], [1.0, 1.0, 1.0]]) - # ] + [gauss, np.array([[0.0, 0.0, 0.0], [1.0, 1.0, 1.0]])], ], ) def test_poisson_bvp_gives_the_correct_laplacian(func, centers):