external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-19 20:28:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2238 from tool4ever/simfix

Browse Source 
Simulation: Fix including unjustified decisions
This commit is contained in:
Anthony Calosa
2023-01-14 07:43:46 +08:00
committed by GitHub
parent 99b9da15b0 c2cea6fb3c
commit e0080f8d43
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
forge-ai/src/main/java/forge/ai/simulation/SimulationController.java
View File

@@ -81,9 +81,11 @@ public class SimulationController {
} }
public void doneEvaluating(Score score) { public void doneEvaluating(Score score) {
if (score.value > bestScore.value) { // if we're here during a deeper level this hasn't been called for the level above yet
// in such case we need to check that this decision has really lead to the improvement in score
if (getLastDecision().initialScore.value < score.value && score.value > bestScore.value) {
bestScore = score; bestScore = score;
bestSequence = currentStack.get(currentStack.size() - 1); bestSequence = getLastDecision();
} }
currentStack.remove(currentStack.size() - 1); currentStack.remove(currentStack.size() - 1);
} }

1
forge-game/src/main/java/forge/game/cost/Cost.java
View File

@@ -254,7 +254,6 @@ public class Cost implements Serializable {
else else
manaParts.append(part).append(" "); manaParts.append(part).append(" ");
} }
} }
if (parsedMana == null && (manaParts.length() > 0 || xCantBe0)) { if (parsedMana == null && (manaParts.length() > 0 || xCantBe0)) {