Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Time handling improvements #8

Merged
merged 5 commits into from
Jul 29, 2021
Merged

Time handling improvements #8

merged 5 commits into from
Jul 29, 2021

Conversation

eduherminio
Copy link
Member

  • Improve time management handling.
    • Use more time at the 'beginning' of the games, where it takes more time to reach the same depth, when no 'moves-to-go' are provided (FirstTimeLimitWhenNoMovesToGoProvided , SecondTimeLimitWhenNoMovesToGoProvided added).
      We already do something similar with KeyMovesBeforeMovesToGo, CoefficientBeforeKeyMovesBeforeMovesToGo and CoefficientAfterKeyMovesBeforeMovesToGo when 'moves-to-go' are provided.
    • Add a minimum MinDepth used in each of the moves, avoiding the situation of making moves with depth 3 in the middle of the game. 5 seems the highest value we can go for right now.
  • Add an 'absolute' CancellationToken so that 'stop' commands act immediately, even if MinDepth hasn't been reached.
  • Remove debug logs from console.
  • Add badges to readme.
  • Remove time clamping, as a consequence of that min depth.

* Use more time at the 'beginning' of the games, where it takes more time to reach the same depth, when no 'moves-to-go' are provided (FirstTimeLimitWhenNoMovesToGoProvided , SecondTimeLimitWhenNoMovesToGoProvided added).
  We already do something similar with KeyMovesBeforeMovesToGo, CoefficientBeforeKeyMovesBeforeMovesToGo and CoefficientAfterKeyMovesBeforeMovesToGo when 'moves-to-go' are provided.
* Add a minimum MinDepth used in each of the moves, avoiding the situation of making moves with depth 3 in the middle of the game. 5 seems the highest value we can go for right now.
…ely, even if MinDepth hasn't been reached.

Remove debug logs from console.
Add badgest to readme.
@eduherminio eduherminio merged commit c7cbaf4 into main Jul 29, 2021
@eduherminio eduherminio deleted the time-handling-improvements branch July 29, 2021 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant