From 3b9e8d6ef69b3427c1559379ecc19c7ab78014ce Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 05:40:34 -0400 Subject: [PATCH 01/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 836550fd1e6..253222de606 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -154,7 +154,7 @@ def test_creation(self): self.assertEqual(Sn.sig[2].have_color, 0) self.assertEqual(Sn.sig[2].npoints, 3) self.assertEqual(Sn.sig[2].oclass, 9) - self.assertTrue(abs(Sn.sig[2].mean[0] - 6.7) < 0.1) + self.assertLess(abs(Sn.sig[2].mean[0] - 6.7), 0.1) self.assertTrue(abs(Sn.sig[2].mean[1] - 8.3) < 0.1) self.assertTrue(abs(Sn.sig[2].var[0][0] - 0.043) < 0.01) self.assertTrue(abs(Sn.sig[2].var[1][1] - 0.3) < 0.1) From 7c3336b1e7d79d35c9f2608255f17cde58f4c9fe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 05:38:42 -0400 Subject: [PATCH 02/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- lib/imagery/testsuite/test_imagery_signature_management.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/imagery/testsuite/test_imagery_signature_management.py b/lib/imagery/testsuite/test_imagery_signature_management.py index 1bc2aecf6df..afedf2b95c9 100644 --- a/lib/imagery/testsuite/test_imagery_signature_management.py +++ b/lib/imagery/testsuite/test_imagery_signature_management.py @@ -1007,7 +1007,7 @@ def test_multiple_sigs_multiple_mapsets(self): shutil.rmtree(sig_dir1) shutil.rmtree(sig_dir2) # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 2) + self.assertGreaterEqual(ret, 2) ret_list = list(map(utils.decode, sig_list[:ret])) self.assertIn(golden[0], ret_list) self.assertIn(golden[1], ret_list) From 84b4a8ba8d5300a2ffe76e8c8c86920908fce942 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:07:51 -0400 Subject: [PATCH 03/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- python/grass/pygrass/vector/testsuite/test_geometry.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/grass/pygrass/vector/testsuite/test_geometry.py b/python/grass/pygrass/vector/testsuite/test_geometry.py index b29557f086d..f432df7675b 100644 --- a/python/grass/pygrass/vector/testsuite/test_geometry.py +++ b/python/grass/pygrass/vector/testsuite/test_geometry.py @@ -81,7 +81,7 @@ def test_eq(self): point0 = Point(0, 0) point1 = Point(1, 0) self.assertFalse(point0 == point1) - self.assertFalse(point0 == (1, 0)) + self.assertNotEqual(point0, (1, 0)) self.assertTrue(point0 == point0) # noqa: PLR0124 # pylint: disable=R0124 self.assertTrue(point0 == (0, 0)) From 306be7beca24e4a15bfa0111747f7bf290d02d3f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:04:54 -0400 Subject: [PATCH 04/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.svm.train/testsuite/test_i_svm_train.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.svm.train/testsuite/test_i_svm_train.py b/imagery/i.svm.train/testsuite/test_i_svm_train.py index af2ab843172..43399bbe8cb 100644 --- a/imagery/i.svm.train/testsuite/test_i_svm_train.py +++ b/imagery/i.svm.train/testsuite/test_i_svm_train.py @@ -299,7 +299,7 @@ def test_rescaling(self): self.assertTrue(float(R) <= 2) M, R = lines[1].strip().split(" ") self.assertTrue(float(M) > -1 and float(M) < 1) - self.assertTrue(float(R) <= 20) + self.assertLessEqual(float(R), 20) @unittest.skipIf(shutil.which("i.svm.train") is None, "i.svm.train not found.") def test_fail_on_empty_raster(self): From 2dd313b4f2ae4770c3e2e08353365723b145d890 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:04:26 -0400 Subject: [PATCH 05/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- lib/imagery/testsuite/test_imagery_signature_management.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/imagery/testsuite/test_imagery_signature_management.py b/lib/imagery/testsuite/test_imagery_signature_management.py index afedf2b95c9..37b336b415f 100644 --- a/lib/imagery/testsuite/test_imagery_signature_management.py +++ b/lib/imagery/testsuite/test_imagery_signature_management.py @@ -985,7 +985,7 @@ def test_multiple_sigs_multiple_mapsets(self): # As temporary mapset is not in the search path, there must be # at least one sig file present # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 1) + self.assertGreaterEqual(ret, 1) ret_list = list(map(utils.decode, sig_list[:ret])) golden = ( f"{rnd_sig1}@{self.rnd_mapset_name}", From 0db197750fc435c7fdbe0871a7a15d9f305c4f90 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:23:23 -0400 Subject: [PATCH 06/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 253222de606..d4590e5e34b 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -156,7 +156,7 @@ def test_creation(self): self.assertEqual(Sn.sig[2].oclass, 9) self.assertLess(abs(Sn.sig[2].mean[0] - 6.7), 0.1) self.assertTrue(abs(Sn.sig[2].mean[1] - 8.3) < 0.1) - self.assertTrue(abs(Sn.sig[2].var[0][0] - 0.043) < 0.01) + self.assertLess(abs(Sn.sig[2].var[0][0] - 0.043), 0.01) self.assertTrue(abs(Sn.sig[2].var[1][1] - 0.3) < 0.1) From 43332e3147afb0c26655be8abf32d20e86d41227 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:24:15 -0400 Subject: [PATCH 07/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- temporal/t.rename/testsuite/test_t_rename.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/temporal/t.rename/testsuite/test_t_rename.py b/temporal/t.rename/testsuite/test_t_rename.py index 13b94f411a4..b65d22af342 100755 --- a/temporal/t.rename/testsuite/test_t_rename.py +++ b/temporal/t.rename/testsuite/test_t_rename.py @@ -140,7 +140,7 @@ def test_rename(self): print(gs.read_command("t.info", type="strds", input="precip_abs6")) self.assertEqual(strds.metadata.get_description(), "This is the 3 test strds") - self.assertTrue("t.rename" in strds.metadata.get_command()) + self.assertIn("t.rename", strds.metadata.get_command()) def test_rename_with_overwrite(self): """Test renaming a space time raster dataset while overwriting @@ -159,7 +159,7 @@ def test_rename_with_overwrite(self): print(gs.read_command("t.info", type="strds", input="precip_abs2")) self.assertEqual(strds.metadata.get_description(), "This is the 1 test strds") - self.assertTrue("t.rename" in strds.metadata.get_command()) + self.assertIn("t.rename", strds.metadata.get_command()) def test_fail_overwrite_without_flag(self): """Test that renaming a space time raster dataset fails if the From f83184b759144fa956e3bfe2cfeb1d9e5d7a027b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:25:13 -0400 Subject: [PATCH 08/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- .../testsuite/test_r_smooth_edgepreserve.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py index 28b061c8ab4..1c5360f3dac 100644 --- a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py +++ b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py @@ -155,7 +155,7 @@ def test_exp_d_ram(self): function="exp", quiet=True, ) - self.assertTrue(raster_info(out_map_d)["datatype"] == "DCELL") + self.assertEqual(raster_info(out_map_d)["datatype"], "DCELL") self.assertRastersEqual(out_map_d, self.ref_exp_f, precision=0.1) def test_quad_ram(self): From f62e09d59a33f9f0f84f7f4efcd62b8af30755e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:26:29 -0400 Subject: [PATCH 09/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- lib/init/testsuite/test_grass_tmp_mapset.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/init/testsuite/test_grass_tmp_mapset.py b/lib/init/testsuite/test_grass_tmp_mapset.py index 8ff92843d25..955074b7143 100644 --- a/lib/init/testsuite/test_grass_tmp_mapset.py +++ b/lib/init/testsuite/test_grass_tmp_mapset.py @@ -98,8 +98,9 @@ def test_mapset_deleted(self): [self.executable, "--tmp-mapset", self.location, "--exec", "g.proj", "-p"] ) for directory in os.listdir(self.location): - self.assertTrue( - directory in self.subdirs, + self.assertIn( + directory, + self.subdirs, msg="Directory {directory} should have been deleted".format(**locals()), ) From eb46ebe6f3c29dad6ca575221fd7e836edcc2e2f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:28:53 -0400 Subject: [PATCH 10/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- python/grass/pygrass/vector/testsuite/test_geometry.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/grass/pygrass/vector/testsuite/test_geometry.py b/python/grass/pygrass/vector/testsuite/test_geometry.py index f432df7675b..6e164ca7779 100644 --- a/python/grass/pygrass/vector/testsuite/test_geometry.py +++ b/python/grass/pygrass/vector/testsuite/test_geometry.py @@ -83,7 +83,7 @@ def test_eq(self): self.assertFalse(point0 == point1) self.assertNotEqual(point0, (1, 0)) self.assertTrue(point0 == point0) # noqa: PLR0124 # pylint: disable=R0124 - self.assertTrue(point0 == (0, 0)) + self.assertEqual(point0, (0, 0)) def test_repr(self): """Test __eq__""" From 23a7082e424c063dceccc7fd566b6f90f7fb463e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:38:47 -0400 Subject: [PATCH 11/23] Potential fixes for 5 code quality findings (#16) * Apply suggested fix to lib/imagery/testsuite/test_imagery_signature_management.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> * Apply suggested fix to lib/imagery/testsuite/test_imagery_signature_management.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> * Apply suggested fix to lib/imagery/testsuite/test_imagery_signature_management.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> * Apply suggested fix to lib/imagery/testsuite/test_imagery_signature_management.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> * Apply suggested fix to lib/imagery/testsuite/test_imagery_signature_management.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> * Apply suggestion from @echoix --------- Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> --- .../testsuite/test_imagery_signature_management.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/imagery/testsuite/test_imagery_signature_management.py b/lib/imagery/testsuite/test_imagery_signature_management.py index 37b336b415f..84b436b0422 100644 --- a/lib/imagery/testsuite/test_imagery_signature_management.py +++ b/lib/imagery/testsuite/test_imagery_signature_management.py @@ -1032,7 +1032,7 @@ def test_multiple_sigsets_multiple_mapsets(self): # As temporary mapset is not in the search path, there must be # at least one sig file present # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 1) + self.assertGreaterEqual(ret, 1) ret_list = list(map(utils.decode, sig_list[:ret])) golden = ( f"{rnd_sig1}@{self.rnd_mapset_name}", @@ -1054,7 +1054,7 @@ def test_multiple_sigsets_multiple_mapsets(self): shutil.rmtree(sig_dir1) shutil.rmtree(sig_dir2) # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 2) + self.assertGreaterEqual(ret, 2) ret_list = list(map(utils.decode, sig_list[:ret])) self.assertIn(golden[0], ret_list) self.assertIn(golden[1], ret_list) @@ -1080,7 +1080,7 @@ def test_multiple_libsvms_multiple_mapsets(self): # As temporary mapset is not in the search path, there must be # at least one sig file present # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 1) + self.assertGreaterEqual(ret, 1) ret_list = list(map(utils.decode, sig_list[:ret])) golden = ( f"{rnd_sig1}@{self.rnd_mapset_name}", @@ -1102,7 +1102,7 @@ def test_multiple_libsvms_multiple_mapsets(self): shutil.rmtree(sig_dir1) shutil.rmtree(sig_dir2) # There could be more sigs if this is not an empty mapset - self.assertTrue(ret >= 2) + self.assertGreaterEqual(ret, 2) ret_list = list(map(utils.decode, sig_list[:ret])) self.assertIn(golden[0], ret_list) self.assertIn(golden[1], ret_list) From e7092160df5e8476c74fb83a3f680fbaf2d1e507 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:39:13 -0400 Subject: [PATCH 12/23] Apply suggested fix to raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> --- .../testsuite/test_r_smooth_edgepreserve.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py index 1c5360f3dac..22db15e200d 100644 --- a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py +++ b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py @@ -132,7 +132,7 @@ def test_exp_f_ram(self): function="exp", quiet=True, ) - self.assertTrue(raster_info(out_map_f)["datatype"] == "FCELL") + self.assertEqual(raster_info(out_map_f)["datatype"], "FCELL") self.assertRastersEqual(out_map_f, self.ref_exp_f, precision=0.1) def test_exp_d_ram(self): From 6bb4a65ebb5836e017539c17823ae338f23c0fd8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 06:39:14 -0400 Subject: [PATCH 13/23] Apply suggested fix to raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> --- .../testsuite/test_r_smooth_edgepreserve.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py index 22db15e200d..6102a0283af 100644 --- a/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py +++ b/raster/r.smooth.edgepreserve/testsuite/test_r_smooth_edgepreserve.py @@ -109,7 +109,7 @@ def test_exp_i_ram(self): function="exp", quiet=True, ) - self.assertTrue(raster_info(out_map_i)["datatype"] == "CELL") + self.assertEqual(raster_info(out_map_i)["datatype"], "CELL") self.assertRastersEqual(out_map_i, self.ref_exp_i) def test_exp_f_ram(self): From 2a5b169ed31dce54b9a4be3f9dc4206c5aeb5896 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:31:11 -0400 Subject: [PATCH 14/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index d4590e5e34b..e873089ab60 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -146,7 +146,7 @@ def test_creation(self): self.assertEqual(Sn.sig[1].npoints, 4) self.assertEqual(Sn.sig[1].oclass, 6) self.assertTrue(abs(Sn.sig[1].mean[0] - 8.9) < 0.1) - self.assertTrue(abs(Sn.sig[1].mean[1] - 1.5) < 0.1) + self.assertLess(abs(Sn.sig[1].mean[1] - 1.5), 0.1) self.assertTrue(abs(Sn.sig[1].var[0][0] - 0.003) < 0.001) self.assertTrue(abs(Sn.sig[1].var[1][1] - 0.3) < 0.1) # 9 From 3c40ad75b96c5984e4dd535d6d771c44abb1d1fe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:33:25 -0400 Subject: [PATCH 15/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.svm.predict/testsuite/test_i_svm_predict.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/imagery/i.svm.predict/testsuite/test_i_svm_predict.py b/imagery/i.svm.predict/testsuite/test_i_svm_predict.py index 3662124f45d..04c52430d59 100644 --- a/imagery/i.svm.predict/testsuite/test_i_svm_predict.py +++ b/imagery/i.svm.predict/testsuite/test_i_svm_predict.py @@ -167,9 +167,9 @@ def test_semantic_label_mismatch2(self): self.tmp_rasts.append(rast) self.assertModuleFail(isvm) self.assertTrue(isvm.outputs.stderr) - self.assertTrue( - "Signature band count: 2, imagery group band count: 3" - in isvm.outputs.stderr + self.assertIn( + "Signature band count: 2, imagery group band count: 3", + isvm.outputs.stderr, ) @unittest.skipIf(shutil.which("i.svm.predict") is None, "i.svm.predict not found.") From 7d52dc26f9984a2786a30bd07fa4cbf93ee9e9af Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:35:15 -0400 Subject: [PATCH 16/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index e873089ab60..5c214e390e3 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -147,7 +147,7 @@ def test_creation(self): self.assertEqual(Sn.sig[1].oclass, 6) self.assertTrue(abs(Sn.sig[1].mean[0] - 8.9) < 0.1) self.assertLess(abs(Sn.sig[1].mean[1] - 1.5), 0.1) - self.assertTrue(abs(Sn.sig[1].var[0][0] - 0.003) < 0.001) + self.assertLess(abs(Sn.sig[1].var[0][0] - 0.003), 0.001) self.assertTrue(abs(Sn.sig[1].var[1][1] - 0.3) < 0.1) # 9 self.assertEqual(Sn.sig[2].status, 1) From cd7e999e1d8e03809959aeedf0af1ac90c17544b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:35:47 -0400 Subject: [PATCH 17/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 5c214e390e3..6b3df0ce736 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -137,7 +137,7 @@ def test_creation(self): self.assertEqual(Sn.sig[0].npoints, 4) self.assertEqual(Sn.sig[0].oclass, 1) self.assertTrue(abs(Sn.sig[0].mean[0] - 2.9) < 0.1) - self.assertTrue(abs(Sn.sig[0].mean[1] - 7.0) < 0.1) + self.assertLess(abs(Sn.sig[0].mean[1] - 7.0), 0.1) self.assertTrue(abs(Sn.sig[0].var[0][0] - 0.03) < 0.01) self.assertTrue(abs(Sn.sig[0].var[1][1] - 0.6) < 0.1) # 6 From 0d5f584525f639198cb8697d6d739c4ba1e7c68a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:37:20 -0400 Subject: [PATCH 18/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.svm.train/testsuite/test_i_svm_train.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.svm.train/testsuite/test_i_svm_train.py b/imagery/i.svm.train/testsuite/test_i_svm_train.py index 43399bbe8cb..cd8fe518874 100644 --- a/imagery/i.svm.train/testsuite/test_i_svm_train.py +++ b/imagery/i.svm.train/testsuite/test_i_svm_train.py @@ -296,7 +296,7 @@ def test_rescaling(self): lines = rf.readlines() M, R = lines[0].strip().split(" ") self.assertTrue(float(M) > -1 and float(M) < 1) - self.assertTrue(float(R) <= 2) + self.assertLessEqual(float(R), 2) M, R = lines[1].strip().split(" ") self.assertTrue(float(M) > -1 and float(M) < 1) self.assertLessEqual(float(R), 20) From ab7f01e7ea468ee5597c217c46fa81ec98155470 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 07:40:17 -0400 Subject: [PATCH 19/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 6b3df0ce736..4296567fbd6 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -136,7 +136,7 @@ def test_creation(self): self.assertEqual(Sn.sig[0].have_color, 0) self.assertEqual(Sn.sig[0].npoints, 4) self.assertEqual(Sn.sig[0].oclass, 1) - self.assertTrue(abs(Sn.sig[0].mean[0] - 2.9) < 0.1) + self.assertLess(abs(Sn.sig[0].mean[0] - 2.9), 0.1) self.assertLess(abs(Sn.sig[0].mean[1] - 7.0), 0.1) self.assertTrue(abs(Sn.sig[0].var[0][0] - 0.03) < 0.01) self.assertTrue(abs(Sn.sig[0].var[1][1] - 0.6) < 0.1) From e3d67adb396063d3630e72cfbce3f8ecd4371c8a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 10:05:32 -0400 Subject: [PATCH 20/23] Apply suggested fix to imagery/i.gensig/testsuite/test_i_gensig.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 4296567fbd6..95286fe7a72 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -155,9 +155,9 @@ def test_creation(self): self.assertEqual(Sn.sig[2].npoints, 3) self.assertEqual(Sn.sig[2].oclass, 9) self.assertLess(abs(Sn.sig[2].mean[0] - 6.7), 0.1) - self.assertTrue(abs(Sn.sig[2].mean[1] - 8.3) < 0.1) + self.assertLess(abs(Sn.sig[2].mean[1] - 8.3), 0.1) self.assertLess(abs(Sn.sig[2].var[0][0] - 0.043), 0.01) - self.assertTrue(abs(Sn.sig[2].var[1][1] - 0.3) < 0.1) + self.assertLess(abs(Sn.sig[2].var[1][1] - 0.3), 0.1) if __name__ == "__main__": From 20335600e4038a1ed4c06cea504e8047733ab640 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 10:05:32 -0400 Subject: [PATCH 21/23] Apply suggested fix to imagery/i.gensig/testsuite/test_i_gensig.py from Copilot Autofix Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index 95286fe7a72..eeef404894d 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -138,17 +138,17 @@ def test_creation(self): self.assertEqual(Sn.sig[0].oclass, 1) self.assertLess(abs(Sn.sig[0].mean[0] - 2.9), 0.1) self.assertLess(abs(Sn.sig[0].mean[1] - 7.0), 0.1) - self.assertTrue(abs(Sn.sig[0].var[0][0] - 0.03) < 0.01) + self.assertLess(abs(Sn.sig[0].var[0][0] - 0.03), 0.01) self.assertTrue(abs(Sn.sig[0].var[1][1] - 0.6) < 0.1) # 6 self.assertEqual(Sn.sig[1].status, 1) self.assertEqual(Sn.sig[1].have_color, 0) self.assertEqual(Sn.sig[1].npoints, 4) self.assertEqual(Sn.sig[1].oclass, 6) - self.assertTrue(abs(Sn.sig[1].mean[0] - 8.9) < 0.1) + self.assertLess(abs(Sn.sig[1].mean[0] - 8.9), 0.1) self.assertLess(abs(Sn.sig[1].mean[1] - 1.5), 0.1) self.assertLess(abs(Sn.sig[1].var[0][0] - 0.003), 0.001) - self.assertTrue(abs(Sn.sig[1].var[1][1] - 0.3) < 0.1) + self.assertLess(abs(Sn.sig[1].var[1][1] - 0.3), 0.1) # 9 self.assertEqual(Sn.sig[2].status, 1) self.assertEqual(Sn.sig[2].have_color, 0) From d70ad3e586248016cf0914834af2eb53d2395044 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 10:32:22 -0400 Subject: [PATCH 22/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- python/grass/pygrass/vector/testsuite/test_geometry_attrs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/grass/pygrass/vector/testsuite/test_geometry_attrs.py b/python/grass/pygrass/vector/testsuite/test_geometry_attrs.py index 8fa198bea0d..2e19ca61c3b 100644 --- a/python/grass/pygrass/vector/testsuite/test_geometry_attrs.py +++ b/python/grass/pygrass/vector/testsuite/test_geometry_attrs.py @@ -44,7 +44,7 @@ def test_getitem(self): with self.assertRaises(ValueError) as cm: self.attrs["not_existing_column_name"] - self.assertTrue("not_existing_column_name" in str(cm.exception)) + self.assertIn("not_existing_column_name", str(cm.exception)) def test_setitem(self): """Test __setitem__ magic method""" From 5bc97651d4d45a3806c54e080664b7c5491f5cbe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Edouard=20Choini=C3=A8re?= <27212526+echoix@users.noreply.github.com> Date: Tue, 12 May 2026 22:14:52 -0400 Subject: [PATCH 23/23] Fix for Imprecise assert Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> --- imagery/i.gensig/testsuite/test_i_gensig.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imagery/i.gensig/testsuite/test_i_gensig.py b/imagery/i.gensig/testsuite/test_i_gensig.py index eeef404894d..cffea461894 100644 --- a/imagery/i.gensig/testsuite/test_i_gensig.py +++ b/imagery/i.gensig/testsuite/test_i_gensig.py @@ -139,7 +139,7 @@ def test_creation(self): self.assertLess(abs(Sn.sig[0].mean[0] - 2.9), 0.1) self.assertLess(abs(Sn.sig[0].mean[1] - 7.0), 0.1) self.assertLess(abs(Sn.sig[0].var[0][0] - 0.03), 0.01) - self.assertTrue(abs(Sn.sig[0].var[1][1] - 0.6) < 0.1) + self.assertLess(abs(Sn.sig[0].var[1][1] - 0.6), 0.1) # 6 self.assertEqual(Sn.sig[1].status, 1) self.assertEqual(Sn.sig[1].have_color, 0)