Skip to content

Commit c05e48b

Browse files
committed
remove mvv-lva
1 parent 0e20284 commit c05e48b

File tree

2 files changed

+16
-17
lines changed

2 files changed

+16
-17
lines changed

src/engine/move_generation/mod.rs

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ use chess::Board;
99
use chess::ChessMove;
1010
use chess::EMPTY;
1111
use chess::MoveGen;
12-
use heuristics::mvv_lva_score;
1312

1413
use crate::evaluation::bitboards::CENTER_4;
1514
use crate::evaluation::bitboards::CENTER_16;
@@ -45,18 +44,18 @@ pub fn prio_iterator(mut mgen: MoveGen, pos: &Board, prio: &[ChessMove]) -> Orde
4544
!EMPTY,
4645
];
4746

48-
// collect capture moves and sort by mvv-lva
49-
mgen.set_iterator_mask(*pos.color_combined(!pos.side_to_move()));
50-
51-
let mut prio_moves: Vec<ChessMove> = mgen.by_ref().collect();
52-
prio_moves.sort_by_cached_key(|mv| mvv_lva_score(pos, mv));
53-
54-
prio_moves.extend_from_slice(prio);
47+
// // collect capture moves and sort by mvv-lva
48+
// mgen.set_iterator_mask(*pos.color_combined(!pos.side_to_move()));
49+
//
50+
// let mut prio_moves: Vec<ChessMove> = mgen.by_ref().collect();
51+
// prio_moves.sort_by_cached_key(|mv| mvv_lva_score(pos, mv));
52+
//
53+
// prio_moves.extend_from_slice(prio);
5554

5655
mgen.set_iterator_mask(masks[0]);
5756

5857
OrderedMoves {
59-
prio_moves,
58+
prio_moves: prio.to_vec(),
6059
mgen,
6160
masks,
6261
cur_mask: 0,

src/engine/search/negamax.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -96,14 +96,6 @@ pub fn negamax(
9696
opts: &Opts,
9797
table: &ShareImpl,
9898
) -> SearchResult {
99-
// the initial move generator
100-
let mut base_gen = MoveGen::new_legal(&pos.chessboard);
101-
// slice for already generated moves.
102-
let mut pre_generated: [Option<ChessMove>; 4] = [None; 4];
103-
104-
// we are mated!
105-
let out_of_moves = base_gen.len() == 0;
106-
10799
optlog!(search;trace;"ng: {pos}, td: {to_depth:?}, a: {alpha:?}, b: {beta:?}");
108100

109101
let current_hash = pos.chessboard.get_hash();
@@ -125,6 +117,14 @@ pub fn negamax(
125117
};
126118
}
127119

120+
// the initial move generator
121+
let mut base_gen = MoveGen::new_legal(&pos.chessboard);
122+
// slice for already generated moves.
123+
let mut pre_generated: [Option<ChessMove>; 4] = [None; 4];
124+
125+
// we are mated!
126+
let out_of_moves = base_gen.len() == 0;
127+
128128
/* source: https://en.wikipedia.org/wiki/Negamax */
129129
let alpha_orig = alpha;
130130
if opts.use_tt {

0 commit comments

Comments
 (0)