Skip to content
Permalink
Browse files

Simplify the code. Take 7, fixed. Bench: 3675826.

  • Loading branch information...
Stefano80 committed Mar 2, 2019
1 parent aa3af81 commit 562dd9122a28c39db45653eba11b7d78ee16ce5c
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/search.cpp
@@ -1090,7 +1090,7 @@ namespace {
resultNN = (value > alpha) + (value >= beta) - 1;

if (trainNN && resultNN != prediction){
// For some reason, in 0.5% of the cases, the prediction here is not the same as before. ToDo.
// For some reason, in 0.5% of the cases, the prediction is not the same as before. ToDo.
LMRnetwork.train(testNN, resultNN, prediction, 0.1);
trainNN = false;
}

3 comments on commit 562dd91

@Vizvezdenec

This comment has been minimized.

Copy link

replied Mar 2, 2019

I hope that everything is OK with the code because I'm not able to read 200 new lines and see if it has no mistakes :P
Good luck!

@Stefano80

This comment has been minimized.

Copy link
Owner Author

replied Mar 2, 2019

Hi, actually it is a bit buggy, I got it from another repo. Now that I understand that it could work, I will have to go and rewrite it from scratch in SF style.

But it is not malicious...

@Vizvezdenec

This comment has been minimized.

Copy link

replied Mar 2, 2019

I recommend you to go for 1/6 throughput for LTCs that are "not for committal" so they wouldn't drag 85% of fishtest workers :)

Please sign in to comment.
You can’t perform that action at this time.