-
Notifications
You must be signed in to change notification settings - Fork 10
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
Build transposition table #205
Conversation
This reverts commit 47e39ba.
72e9a7a
to
863c8b4
Compare
863c8b4
to
aa7bb1b
Compare
This reverts commit f9e6eea.
endgame/alphabeta/alphabeta.go
Outdated
@@ -366,6 +391,7 @@ func (s *Solver) Solve(plies int) (float32, []*move.Move, error) { | |||
|
|||
tstart := time.Now() | |||
s.mmCount = 0 | |||
s.zobrist.Initialize(s.game.Board().Dim(), alphabet.MaxAlphabetSize+1, alphabet.MaxAlphabetSize+1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for norwegian, the second argument to this function is smaller than the third argument. This is because Norwegian can make the blank a bunch of letters that are not actually in its tile distribution. I don't think I have this argument readily available but it should be kept in mind that we need to modify it for Norwegian.
zobrist/hash.go
Outdated
z.posTable[i] = make([]int64, numtiles+numblankletters) | ||
for j := 0; j < numtiles+numblankletters; j++ { | ||
z.posTable[i][j] = int64(frand.Uint64n(bignum)) | ||
z.posTable[i] = make([]uint64, 256) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens if you use numtiles+numblankletters
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was observing index out of bounds accesses... spent the better part of an hour trying to make it work before trying 256
instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've committed this suggestion.
f62cb7a
to
993a8cc
Compare
dbb07db
to
616b3c4
Compare
575306f
to
779376c
Compare
No description provided.