Skip to content

Commit

Permalink
depth > 4
Browse files Browse the repository at this point in the history
  • Loading branch information
MJZ1977 committed Jan 9, 2019
1 parent 6f9e202 commit 329d868
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/search.cpp
Expand Up @@ -677,7 +677,7 @@ namespace {
return ttValue;
}

if (depth > 5 * ONE_PLY && !excludedMove && PvNode)
if (depth > 4 * ONE_PLY && !excludedMove && PvNode)
pvHit = true;

// Step 5. Tablebases probe
Expand Down

5 comments on commit 329d868

@Vizvezdenec
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should make PR for your test MJZ :)

@MJZ1977
Copy link
Owner Author

@MJZ1977 MJZ1977 commented on 329d868 Jan 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You mean priority ?
I will set to 1, hope nobody will be offensed :-)

@snicolet
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is parameter tuning at priority 1 for a patch which has not yet been committed :-/

@vondele
Copy link

@vondele vondele commented on 329d868 Jan 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MJZ1977 no PR is 'pull request'. I've set priority back to normal. Prio 1 should be used exceptionally, typically only by maintainers, and sometimes to run threaded tests that require one of the few e.g. 30 core machines.

@MJZ1977
Copy link
Owner Author

@MJZ1977 MJZ1977 commented on 329d868 Jan 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, no problem.
I stopped the tests because the results are clear now. My objective was just to submit the pull request with the best parameters.

Please sign in to comment.