From d4b09f39ad03e2e17074e6182274f4ff10ea91c9 Mon Sep 17 00:00:00 2001 From: austinio7116 Date: Wed, 28 Feb 2018 12:10:53 +0000 Subject: [PATCH] Refactoring of card-based deck generation to share the same generation code so that the Modern/Standard deck generation gets the benefit of improvements made in the Commander deck generation code - also fixed a few bugs with land counts, generic mana cards with a colour identity. Reduced the minimum number of connections in the matrix required to include cards in the card-based deckgen list to get more cards included in the model. --- .../src/main/java/forge/limited/CardThemedDeckBuilder.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java b/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java index 6c4298ab34e..c290875b2e8 100644 --- a/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java +++ b/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java @@ -663,9 +663,9 @@ public class CardThemedDeckBuilder extends DeckGeneratorBase { List landsToAdd = new ArrayList<>(); int minBasics;//Keep a minimum number of basics to ensure playable decks if(colors.isMonoColor()){ - minBasics=Math.round((r.nextInt(15)+9)*new Float(targetSize)/new Float(60)); + minBasics=Math.round((r.nextInt(15)+9)*((float) targetSize) / 60); }else{ - minBasics=Math.round((r.nextInt(8)+6)*new Float(targetSize)/new Float(60)); + minBasics=Math.round((r.nextInt(8)+6)*((float) targetSize) / 60); }