Draft
Conversation
Contributor
Author
|
I thought of a solution I like. Unify the facetoface and aigame views, by implementing the AI stuff as i have here, in facetoface, so it's using compose, but keep the two shortcuts. That way, its simple when it needs to be, but also you have the full flexibility to e.g. load an sgf and continue playthrough with katago |
0f635a1 to
d3e55f8
Compare
Draft
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Last major modification for now I think, managed to salvage this one too. The SGF part is still broken so drafted until I can fix that or separate it out, cause the code could be used if adapted to your Position.kt rewrite.
Added optional AI detail on hints
Two new settings options for AI fine-tuning (maxVisits is a log-scale)
AI game supports playing games without the AI, or with AI on both sides, dynamically. This was obsoleted a bit when you added the facetoface screen. Will need to rethink, but unsure where to go with that