Merge branch 'master' of http://git.winfor.ch/nicolas/bishop

src/bitboard.rs

@@ -85,16 +85,25 @@ pub fn square(b: Bitboard) -> Square {
     b.trailing_zeros() as Square
 }
 
+///
+/// \brief calculates the square index of a square
+/// 
+/// \param file the file of the square with 0 being the a file and 7 being the h file
+/// \param row the row of the square with 0 being row 1 and 7 being row 8
+/// 
 pub fn square_from_indices(file: u8, row: u8) -> Square {
-    (7 - row) * 8 + 7 - file
+    (row) * 8 + 7 - file
 }
 
 pub fn from_indices(file: u8, row: u8) -> Bitboard {
     1_u64 << square_from_indices(file, row)
 }
 
+///
+/// reverse of indices_from_square
+/// 
 pub fn indices_from_square(sq: Square) -> (u8, u8) {
-    (7 - sq % 8, 7 - sq / 8)
+    (7 - sq % 8, sq / 8)
 }
 
 pub fn print_board(b: Bitboard) -> String {

src/engine.rs

@@ -8,6 +8,7 @@ use log::{info};
 use std::time::{Duration, Instant};
 use std::collections::{VecDeque, HashMap};
 use zobrist::ZobristTable;
+use hash::Cache;
 
 use std::thread::sleep_ms;
 use std::sync::mpsc::{Receiver, Sender};
@@ -40,16 +41,12 @@ pub enum SearchInfo {
 pub enum InterfaceMsg {
 }
 
-pub struct HashEntry {
-
-}
-
 pub struct Engine {
     game: Game,
     messages: VecDeque<EngineMsg>,
     r: Receiver<EngineMsg>,
     s: Sender<InterfaceMsg>,
-    hash: HashMap<Game, HashEntry>,
+    hash: Cache,
     zobrist_table: ZobristTable
 }
 
@@ -59,7 +56,7 @@ impl Engine {
             game: Game::default(),
             messages: VecDeque::new(),
             r, s,
-            hash: HashMap::new(),
+            hash: Cache::new(),
             zobrist_table: ZobristTable::new()
         };
         eng.game.zobrist = Some(eng.game.calculate_zobrist(&eng.zobrist_table));

src/game.rs

@@ -77,9 +77,9 @@ impl Game {
 
         {
             let rows = position.split('/');
-            let mut row_index = 0;
+            let mut row_index: i32 = 7;
             for row in rows {
-                if row_index >= 8 {
+                if row_index < 0 {
                     return None;
                 }
                 let mut col_index = 0;
@@ -89,7 +89,7 @@ impl Game {
                         return None;
                     }
 
-                    let bit_to_set = from_indices(col_index, row_index);
+                    let bit_to_set = from_indices(col_index, row_index as _);
                     match c {
                         'p' => { *game.pawns_mut(BLACK) |= bit_to_set },
                         'n' => { *game.knights_mut(BLACK) |= bit_to_set },
@@ -112,7 +112,7 @@ impl Game {
                     }
                     col_index += 1;
                 }
-                row_index += 1;
+                row_index -= 1;
             }
         }
 

src/hash.rs

@@ -18,6 +18,16 @@ pub struct CacheEntry {
     pub value: PosValue
 }
 
+impl CacheEntry {
+    pub fn new_value(depth: i32, value: PosValue) -> Self {
+        CacheEntry {
+            entry_type: EntryType::Value,
+            depth,
+            value
+        }
+    }
+}
+
 pub struct Cache {
     hashmap: HashMap<Game, CacheEntry>,
 }
@@ -39,5 +49,8 @@ impl Cache {
             self.hashmap.remove(&first_key);
         }
         self.hashmap.insert(game_pos.clone(), ce);
+        if self.hashmap.len() % 1024 == 0 {
+            info!("hash contains {} items", self.hashmap.len());
+        }
     }
 }

src/search.rs

@@ -5,8 +5,7 @@ use evaluate::*;
 use log::info;
 use rand::prelude::*;
 use std::collections::HashMap;
-use engine::HashEntry;
+use hash::{CacheEntry, Cache};
-use hash::*;
 
 enum MoveUndo {
     Default {
@@ -128,6 +127,7 @@ pub fn search(game: &Game, sc: &mut SearchControl, hash: &mut Cache, depth: i32)
             return SearchResult::Cancelled(Some((chosen_mov.0, chosen_mov.1)));
         }
         else {
+            hash.cache(game, CacheEntry::new_value(depth, chosen_mov.1));
             return SearchResult::Finished(chosen_mov.0, chosen_mov.1);
         }
     }
@@ -184,7 +184,7 @@ fn negamax(game: &Game, sc: &mut SearchControl, hash: &mut Cache, mut alpha: Pos
             return (beta, false);
         }
         if val > alpha {
-            alpha = (val as f64 * 0.95) as _;
+            alpha = val;//(val as f64 * 0.95) as _;
         }
         //info!(" -> negamaxed {} -> {}", mov.to_string(), depth - 1);
         if val > best {
@@ -194,7 +194,7 @@ fn negamax(game: &Game, sc: &mut SearchControl, hash: &mut Cache, mut alpha: Pos
         }
     }
     //info!("best alpha {}", best);
-    return ((best as f64 * 0.99) as _, false);
+    return ((alpha as f64 * 0.99) as _, false);
 }
 
 fn quiescence_search(game: &Game, si: &mut SearchControl, mut alpha: PosValue, beta: PosValue, depth: i32) -> PosValue {