Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

Delete all ancestors of the tree root after playing a move #914

Open
tommadams opened this issue Oct 9, 2019 · 1 comment
Open

Delete all ancestors of the tree root after playing a move #914

tommadams opened this issue Oct 9, 2019 · 1 comment

Comments

@tommadams
Copy link
Contributor

That way the tree's "root" is actually the real root of the tree.
Also the current implementation of the undo GTP command is completely broken I think because it just changes the root to be its own parent but the parent's visit counts no longer match the counts of its children (because they get pruned).

This would require updating at least GtpClient::HandleUndo and MiniguiGtpClient::ReplaySgf.

@tommadams
Copy link
Contributor Author

The other really important thing is to update the root node's superko_cache.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant