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.

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

Changing komi should cause Lizzie to regenerate analysis #643

Open
lightvector opened this issue Oct 20, 2019 · 8 comments
Open

Changing komi should cause Lizzie to regenerate analysis #643

lightvector opened this issue Oct 20, 2019 · 8 comments

Comments

@lightvector
Copy link

If you do the following:

  • Play through an SGF with Lizzie interactively, causing many moves to get thousands of playouts establishing the winrate.
  • Turn off pondering
  • Change the komi (Game -> Set gameInfo)
  • Go back to the start of the SGF
  • Resume pondering
  • Scroll forward through the SGF to try to analyze with the new komi.

Many of the moves will display prominently incorrect winrates and/or scores, until many seconds or minutes pass on a move so that the bot's search with the new komi finally exceeds the total playouts of the search of the old komi. It is possible to manually clear a move one by one (Analyze -> Clear analysis) but it seems that Lizzie should know that the komi has changed and do this automatically.

Of course, possibly users would be sad if the entire analysis was lost when changing komi and changing it back. If so, one might imagine that alternatively Lizzie only clears the analysis when actually repondering a position after komi is changed instead of clearing all analysis in the SGF immediately when komi is changed. Or perhaps changing komi doesn't clear the analysis at all, instead Lizzie remembers and saves for each move a mapping from komi -> analysis, and only displays the analysis for a particular komi when the gameInfo matches that komi. Or any similar such solution.

Thanks!

@featurecat
Copy link
Owner

@zsalch could you take a look?

@wonsiks
Copy link

wonsiks commented Oct 21, 2019

@lightvector Try change show-bestmoves-by-hold parameter into false in config file.

@zsalch
Copy link
Contributor

zsalch commented Oct 21, 2019

Yes, please try the option by @wonsiks .
And I will try to clear by Lizzie when the komi is changed.

@featurecat
Copy link
Owner

(and engine change). We are thinking that perhaps the move history can store individual stats for each komi/engine.

@featurecat
Copy link
Owner

to make this process slightly easier I added a shortcut in 62d9266 ctrl+A which means you don't have to go through the menu for each move. Due to the poor code structure I haven't been able to figure out how to change analysis on engine/komi change yet. I'll leave that to zsalch.

@lightvector
Copy link
Author

lightvector commented Oct 25, 2019

Disabling show-bestmoves-by-hold when komi is changing to disable the feature entirely is an okay workaround. But eagerly await the more selective change within Lizzie itself to handle komi change intelligently and maybe have per-komi or per-engine stats. :)

@lightvector
Copy link
Author

And a hotkey to clear analysis is great, thanks.

@zsalch
Copy link
Contributor

zsalch commented Nov 4, 2019

@lightvector
I have created a new PR #658. You can try it and let me know if have any questions.

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

No branches or pull requests

4 participants