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

⚡ Force GC collection at the end of Engine constructor and after ucinewgame #685

Merged
merged 1 commit into from Feb 27, 2024

Conversation

eduherminio
Copy link
Member

@eduherminio eduherminio commented Feb 27, 2024

Tested using:

ucinewgame
go depth 15
go depth 15
ucinewgame
ucinewgame

Before:

image

After:

image

NPS looking good in bench
image
image

Test  | perf/force-gc-after-newgame
Elo   | -0.55 +- 2.73 (95%)
SPRT  | 4.0+0.04s Threads=1 Hash=32MB
LLR   | -2.27 (-2.25, 2.89) [0.00, 3.00]
Games | N: 40842 W: 13403 L: 13468 D: 13971
Penta | [1818, 4392, 8026, 4407, 1778]
https://openbench.lynx-chess.com/test/203/
> python .\sprt.py -w 13403 -d 13971 -l 13468 -e0 -5 -e1 0
ELO: -0.553 +- 2.73 [-3.28, 2.18]
LLR: 3.76 [-5.0, 0.0] (-2.94, 2.94)
H1 Accepted

Base automatically changed from perf/remove-staticeval-heap-allocations to main February 27, 2024 00:52
@eduherminio eduherminio marked this pull request as ready for review February 27, 2024 09:59
@eduherminio eduherminio changed the title Force GC collection at the end of Engine constructor and after ucinewgame ⚡ Force GC collection at the end of Engine constructor and after ucinewgame Feb 27, 2024
@eduherminio eduherminio merged commit 57a9b0f into main Feb 27, 2024
27 checks passed
@eduherminio eduherminio deleted the perf/force-gc-after-newgame branch February 27, 2024 09:59
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