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

Fix blank variation returned for minimum index #42

Merged

pdil merged 1 commit into master from bugfix/full-variation-for-min-index

Sep 11, 2024

Member

pdil commented Sep 11, 2024

Passing in .minimum to MoveTree.fullVariation(for:) returned an empty array even if there were valid moves in the move tree.
If this value is passed, it will use the first valid index (.minimum.next) to calculate the moves contained in the variation.

pdil added the bug label

pdil self-assigned this


          Fix blank variation returned for minimum index

fec61ec

pdil force-pushed the bugfix/full-variation-for-min-index branch from 1a69511 to fec61ec Compare

September 11, 2024 19:58

pdil marked this pull request as ready for review

September 11, 2024 20:00

pdil merged commit e420a47 into master

5 checks passed

pdil deleted the bugfix/full-variation-for-min-index branch

September 11, 2024 20:02

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

Labels

bug