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

Template Corrhist #5650

Closed
wants to merge 1 commit into from
Closed

Template Corrhist #5650

wants to merge 1 commit into from

Conversation

xu-shawn
Copy link
Contributor

Avoids duplication of using ... = Stats<int16_t, CORRECTION_HISTORY_LIMIT, COLOR_NB, CORRECTION_HISTORY_SIZE>;

no functional change

Disservin
Disservin requested changes Oct 23, 2024
View reviewed changes
src/movepick.h Outdated Show resolved Hide resolved
src/movepick.h Outdated Show resolved Hide resolved
src/movepick.h Outdated Show resolved Hide resolved
src/movepick.h Outdated Show resolved Hide resolved
@xu-shawn @Disservin
template corrhist
0754df5 
bench 1283457

Co-authored-by: Disservin <[email protected]>
@xu-shawn xu-shawn mentioned this pull request Oct 24, 2024
Closed
@Disservin Disservin closed this in 4a9c980 Oct 31, 2024
Disservin pushed a commit that referenced this pull request Nov 13, 2024
@xu-shawn @Disservin
Move history code to a separate header
c2611ef 
Since no correction histories are ever used inside Movepick, and many
existing histories are closely integrated into search, it might be more
logical to separate them into their own header. PR based on #5650

closes #5652

No functional change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Disservin Disservin Disservin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xu-shawn @Disservin