README.md

@@ -19,34 +19,12 @@ au programme via la commande `./game.py -h`. Voici quelques exemple de
 lancement:
 
 ```shell
-# Lancement de base: les deux joueurs jouent avec le moteur aléatoire et les
-options de base:
+# Lancement de base: les deux joueurs jouent avec le moteur aléatoire:
 ./game.py
 
 # joueur noir humain et joueur blanc MinMax avec une profondeur de 5
-./game.py -be human -we minmax --white-depth-exploration 5
+./game.py -b human -w minmax --depth 5
 ```
-Voici la liste des options :
-
- * `-be` | `--black-player-engine`: moteur utilisé par le joueur avec les pions
-   noirs
- * `-we` | `--white-player-engine`: moteur utilisé par le joueur avec les pions
-   blancs
- * `-bd` | `--black-depth-exploration`: niveau d'eploration de l'arbre de jeu
-   pour le joueur au pions noirs, valable pour les moteurs `minmax` et
-   `alphabeta`
- * `-wd` | `--white-depth-exploration`: niveau d'eploration de l'arbre de jeu
-   pour le joueur au pions noirs, valable pour les moteurs `minmax` et
-   `alphabeta`
- * `-bh` | `--black-heuristic-engine`: moteur heuristique utilisé pour
-   l'exploration de l'arbre de jeu du joueur noir (valable pour les moteur de
-   jeu `minmax` et `alphabeta`)
- * `-wh` | `--black-heuristic-engine`: moteur heuristique utilisé pour
-   l'exploration de l'arbre de jeu du joueur blanc (valable pour les moteur de
-   jeu `minmax` et `alphabeta`)
-
-L'affichage verbeux est activé avec `-V` et les informations de débogage sont 
-affichée avec l'option `-d`.
 
 ## Choix d'implémentation
 

src/classes/Engines.py

@@ -1,15 +1,11 @@
 import random, math
-from .Heuristic import ReversiHeuristic
 
 class PlayerEngine:
     def __init__(self, logger, options: dict = {}):
         # init logger do display informations
         self.logger = logger
         self.options = options
-        self.logger.info("Init engine {}, options:{}".format(
-            self.__class__.__name__,
-            self.options
-        ))
+        self.logger.info("Init engine {}".format(self.__class__.__name__))
 
     def get_move(self, board):
         self.logger.info("engine: {} - player:{}".format(
@@ -92,11 +88,7 @@ class MinmaxPlayerEngine(PlayerEngine):
         move = ''
         if depth == 0:
             leafs +=1
-            if self.options['heuristic'] == 'weight':
-                score = ReversiHeuristic(board).get()
-            else:
-                score = board.heuristique()
-            return score, nodes, leafs
+            return board.heuristique(), nodes, leafs
         
         if friend_move:
             value = -math.inf
@@ -158,11 +150,7 @@ class AlphabetaPlayerEngine(PlayerEngine):
         leafs = 0
         if depth == 0 :
             leafs +=1
-            if self.options['heuristic'] == 'weight':
-                score = ReversiHeuristic(self.logger).get(board)
-            else:
-                score = board.heuristique()
-            return score, nodes, leafs
+            return board.heuristique(), nodes, leafs
 
         if friend_move:
             value = -math.inf

src/classes/Heuristic.py

@@ -1,38 +0,0 @@
-class ReversiHeuristic():
-    def __init__(self, logger, weight = [1, 2, 10, 20]):
-        self.logger = logger
-        self.weight = weight
-
-    def get(self, board):
-        size = board.get_board_size()
-        score = 0
-        weights = self._get_weight(size)
-        for pos_x in range(size):
-            for pos_y in range(size):
-                if board._board[pos_x][pos_y] == board._nextPlayer:
-                    score += weights[pos_x][pos_y]
-                else:
-                    score -= weights[pos_x][pos_y]
-        return score
-        
-    def _get_weight(self, size):
-        w = [[ 0 for _ in range(size)] for _ in range(size)]
-
-        for pos_y in range(size):
-            for pos_x in range(size):
-
-                # Elements in the corner
-                if pos_x in [0, size -1] and pos_y in [0,size - 1]:
-                    w[pos_x][pos_y] = self.weight[3]
-                
-                # Elements on the border
-                elif pos_x in [0, size -1] or pos_y in [0, size -1]:
-                    w[pos_x][pos_y] = self.weight[2]
-                
-                # Element the center
-                elif pos_x in range( int(size // 2 - 2), size // 2 + 2) and pos_y in range( size // 2 - 2, size // 2 + 2):
-                    w[pos_x][pos_y] = self.weight[1]
-
-                else:
-                    w[pos_x][pos_y] = self.weight[0]
-        return w

src/game.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python3
-
 from classes.Reversi import Board
 from classes.Engines import RandomPlayerEngine, HumanPlayerEngine, MinmaxPlayerEngine, AlphabetaPlayerEngine
 import logging as log
@@ -12,45 +11,26 @@ Function to parse command line arguments
 """
 def parse_aguments():
     engines_choices=['random', 'human', 'minmax', 'alphabeta']
-    heuristic_choices=['weight',]
     parser = arg.ArgumentParser('Playing Reversi with (virtual) friend')
 
-    parser.add_argument('-we', '--white-engine',
+    parser.add_argument('-w', '--white-engine',
         choices=engines_choices,
         help='white player engine (random)',
         default='random'
     )
 
-    parser.add_argument('-be', '--black-engine',
+    parser.add_argument('-b', '--black-engine',
         choices=engines_choices,
         help='black player engine (random)',
         default='random'
     )
 
-    parser.add_argument('-bd', '--black-depth-exploration',
-        help='Black player exploration depth (minmax or alphabeta engine)',
+    parser.add_argument('-D', '--depth',
+        help='Minmax exploration depth',
         type=int,
         default=3,
     )
 
-    parser.add_argument('-wd', '--white-depth-exploration',
-        help='White player exploration depth (minmax or alphabeta engine)',
-        type=int,
-        default=3,
-    )
-
-    parser.add_argument('-bh', '--black-heuristic-engine',
-        help='Black player heutistic engine',
-        choices=['board', 'weight'],
-        default='board',
-    )
-
-    parser.add_argument('-wh', '--white-heuristic-engine',
-        help='White player heutistic engine',
-        choices=['board', 'weight'],
-        default='board',
-    )
-
     debug_group = parser.add_mutually_exclusive_group()
     debug_group.add_argument('-V', '--verbose',
         help='Verbose output',
@@ -74,6 +54,7 @@ if __name__ == '__main__':
     print("Stating PyReverso...")
     args = parse_aguments()
     logger = log.getLogger()
+    print(args)
     # Create handler for streaming to tty (stderr / stdout)
     tty_handler = log.StreamHandler()
     tty_handler.setFormatter(CustomFormatter())
@@ -93,14 +74,11 @@ if __name__ == '__main__':
         args.black_engine,
         args.white_engine
         ))
-    wplayer = engines[args.white_engine](logger, {
-        'depth': args.white_depth_exploration,
-        'heuristic': args.white_heuristic_engine
-    })
-    bplayer = engines[args.black_engine](logger, {
-        'depth': args.black_depth_exploration,
-        'heuristic': args.black_heuristic_engine
-    })
+    options = {
+        'depth': args.depth
+    }
+    wplayer = engines[args.white_engine](logger, options)
+    bplayer = engines[args.black_engine](logger, options)
     while ( not game.is_game_over()):
         if game._nextPlayer == 1:
             move = bplayer.get_move(game)