Fix caching in MixedStrategyProfile

tturocy · tturocy · commit a70fc050a2e8 · 2025-12-04T18:09:07.000Z
For mixed strategy profiles which are defined on trees, a behavior strategy profile is computed and cached. This was not correctly being cleared when changing the strategy probabilities, resulting in incorrect output. Closes #616.
diff --git a/src/games/stratmixed.h b/src/games/stratmixed.h
@@ -49,7 +49,11 @@ template <class T> class MixedStrategyProfileRep {
     return m_probs[m_profileIndex.at(p_strategy)];
   }
   /// Returns the probability the strategy is played
-  T &operator[](const GameStrategy &p_strategy) { return m_probs[m_profileIndex.at(p_strategy)]; }
+  T &operator[](const GameStrategy &p_strategy)
+  {
+    InvalidateCache();
+    return m_probs[m_profileIndex.at(p_strategy)];
+  }
   /// Set the strategy of the corresponding player to a pure strategy
   void SetStrategy(const GameStrategy &p_strategy)
   {

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,11 @@ template <class T> class MixedStrategyProfileRep {`
`49`	`49`	`return m_probs[m_profileIndex.at(p_strategy)];`
`50`	`50`	`}`
`51`	`51`	`/// Returns the probability the strategy is played`
`52`		`- T &operator[](const GameStrategy &p_strategy) { return m_probs[m_profileIndex.at(p_strategy)]; }`
	`52`	`+ T &operator[](const GameStrategy &p_strategy)`
	`53`	`+ {`
	`54`	`+ InvalidateCache();`
	`55`	`+ return m_probs[m_profileIndex.at(p_strategy)];`
	`56`	`+ }`
`53`	`57`	`/// Set the strategy of the corresponding player to a pure strategy`
`54`	`58`	`void SetStrategy(const GameStrategy &p_strategy)`
`55`	`59`	`{`