Added a fix for badumdum, but I don't agree with the interpretation.

Drise13 · Jul 16, 2022 · 6f5f02c · 6f5f02c
1 parent 5c32d2b
commit 6f5f02c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/PetsOptimizer/Genes/GeneticEffects.cs b/PetsOptimizer/Genes/GeneticEffects.cs
@@ -126,11 +126,12 @@ public bool DoesMultiplierApplyToForaging(Territory territory)
 public class BadumdumEffect : IForagerGeneEffect
 {
     public IGeneEffect.GeneApplication Application => Regional;
-    public double StrengthMultiplier => 1.5;
+    public double StrengthMultiplier => 1.2;
 
     public bool DoesMultiplierApplyToForaging(Territory territory)
     {
-        return false;
+        // I don't think this is the right interpretation of "all pets above and below", but Lava's math says it does.
+        return true;
     }
 }
 

diff --git a/PetsOptimizer/Territory.cs b/PetsOptimizer/Territory.cs
@@ -91,15 +91,13 @@ public double GetTotalForagePower()
 
                     break;
 
-                case IGeneEffect.GeneApplication.All:
+                case IGeneEffect.GeneApplication.All or IGeneEffect.GeneApplication.Regional:
+                    foreach (var pet in petForagingPower.Keys)
                     {
-                        foreach (var pet in petForagingPower.Keys)
-                        {
-                            petForagingPower[pet] *= foragingMultiplierPet.GeneEffect.StrengthMultiplier;
-                        }
-
-                        break;
+                        petForagingPower[pet] *= foragingMultiplierPet.GeneEffect.StrengthMultiplier;
                     }
+
+                    break;
             }
         }