BugFix: Incorrect ranges in the definition of the WA Field scoring system making targets too large. Spotted by @TomHall2020.

jatkinson1000 · jatkinson1000 · commit e59da86a0179 · 2024-03-10T14:27:14.000+01:00
diff --git a/archeryutils/handicaps/handicap_scheme.py b/archeryutils/handicaps/handicap_scheme.py
@@ -284,7 +284,7 @@ def arrow_score(  # noqa: PLR0912 Too many branches
                 - np.exp(-((((tar_dia / 20.0) + arw_rad) / sig_r) ** 2))
                 - sum(
                     np.exp(-((((n * tar_dia / 10.0) + arw_rad) / sig_r) ** 2))
-                    for n in range(2, 7)
+                    for n in range(1, 6)
                 )
             )
 
diff --git a/archeryutils/handicaps/tests/test_handicaps.py b/archeryutils/handicaps/tests/test_handicaps.py
@@ -340,7 +340,7 @@ def test_different_handicap_systems(
             ("10_zone_6_ring", 7.397557278),
             ("10_zone_5_ring", 7.059965360),
             ("10_zone_5_ring_compound", 6.993772436),
-            ("WA_field", 4.807397627),
+            ("WA_field", 4.115600784),
             ("IFAA_field", 4.265744101),
             ("IFAA_field_expert", 4.021942762),
             ("Beiter_hit_miss", 0.9998380401),

Original file line number	Diff line number	Diff line change
`@@ -284,7 +284,7 @@ def arrow_score( # noqa: PLR0912 Too many branches`
`284`	`284`	`- np.exp(-((((tar_dia / 20.0) + arw_rad) / sig_r) ** 2))`
`285`	`285`	`- sum(`
`286`	`286`	`np.exp(-((((n * tar_dia / 10.0) + arw_rad) / sig_r) ** 2))`
`287`		`- for n in range(2, 7)`
	`287`	`+ for n in range(1, 6)`
`288`	`288`	`)`
`289`	`289`	`)`
`290`	`290`