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.

Sign up for GitHub

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

Jump to bottom

⌛ Use strict soft and hard bounds for time management #447

Closed
wants to merge 2 commits into from
Closed

⌛ Use strict soft and hard bounds for time management #447

wants to merge 2 commits into from

Conversation

eduherminio
Copy link
Member

@eduherminio eduherminio commented Oct 18, 2023 •
edited

Use what seems to be standard soft/hard bound strategy.
I had a soft bound multiplied for an extra coefficient and didn't have hard bound

1783, 0.08

Score of Lynx 1783 vs Lynx 1781 - main: 1128 - 1205 - 1224  [0.489] 3557
...      Lynx 1783 playing White: 754 - 415 - 609  [0.595] 1778
...      Lynx 1783 playing Black: 374 - 790 - 615  [0.383] 1779
...      White vs Black: 1544 - 789 - 1224  [0.606] 3557
Elo difference: -7.5 +/- 9.2, LOS: 5.5 %, DrawRatio: 34.4 %
SPRT: llr -2.25 (-77.9%), lbound -2.25, ubound 2.89 - H0 was accepted

@eduherminio
Use what seems to be standard soft/hard bound strategy.
dc49ab0 
I had a soft bound multiplied for an extra coefficient and didn't have hard bound
@eduherminio
Merge with origin/main
5628306
@eduherminio eduherminio mentioned this pull request Feb 16, 2024
Merged
@eduherminio eduherminio deleted the time-management/soft-hard-bounds branch February 17, 2024 02:59
eduherminio added a commit that referenced this pull request Feb 17, 2024
@eduherminio
⌛ Make soft and hard bounds explicit, simplifying the existing logic (#…
ea44075 
…664)

Reimplementation of #447.
This is an attempt to standarize time management using the hard/soft bound technique, where different values are used to stop the search after each IDDFS iteration (soft bound) and to stop the search at any moment (hard bound).

We were already using this technique via applying `DecisionTimePercentageToStopSearching` coefficient to the hard bound to get the soft one.
Last tm revision was done in #288


I run this non-ref tests, but I'm trully merging this together with #665, I just want to make clear the separation between the implementation and the value tweaking (fixing even, lol)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@eduherminio