kaorahi / lizgoban

Leela Zero & KataGo visualizer
GNU General Public License v3.0
169 stars 28 forks source link

Show KataGo ownership without numbers #74

Closed raylu closed 2 years ago

raylu commented 2 years ago

currently, the app shows ownership with shift+e (or view > ownership) with green and pink numbers in each region (boundaries shown when holding v) like in the screen recording in the readme: screen recording

the 0.7.0-pre2 releases page shows a screenshot with just per-point ownership info in a single color (red) screenshot I'm not sure how to get this mode. (I'm also not sure how to interpret this; shouldn't there be 2 colors?)

assuming this mode isn't currently possible to get, would you accept a PR to turn off the green and pink numbers?

kaorahi commented 2 years ago

The second screenshot is not the ownership but the standard deviation of the ownership. It is shown in the subboard by default.

There is no way to show the ownership without the green/pink numbers at present. Before considering the option to hide them, I'd like to ask...

Have you tried katrain or lizzie? They can show the ownership more beautifully. Do you have a reason why you dare to use lizgoban?

raylu commented 2 years ago

haha, actually I normally use lizzie. I just tried lizgoban yesterday and I've played around with katrain a little previously. lizzie seems like a dead project at this point, so I'm looking to switch to something else

kaorahi commented 2 years ago

My main motivation of this project is not a clone of lizzie but experiments of new ideas on visualizations and gimmicks. If you are just searching for an alternative to lizzie, you may be happier to choose katrain, lizzieyzy, or a patched lizzie (featurecat/lizzie#798). ;)

Would you let me know again when you find a reason to use lizgoban and still want to hide those numbers?

(FYI: the easiest hack to hide them in lizgoban v0.7.0-pre2)

--- a/src/draw_goban.js
+++ b/src/draw_goban.js
@@ -488,7 +488,7 @@ function goban_bg(pausing_p, trial_p, border) {
 function draw_endstate_clusters(boundary_p, unit, idx2coord, g) {
     const style = boundary_p ?
           {black: 'rgba(0,255,255,0.5)', white: 'rgba(255,0,0,0.5)'} :
-          {black: 'rgba(0,255,0,0.2)', white: 'rgba(255,0,255,0.2)'}
+          {black: 'rgba(0,255,0,0)', white: 'rgba(255,0,255,0)'}
     const size = {major: 3, minor: 2}
     const past_p = past_endstate_p(boundary_p)
     const cs = (past_p ? R.prev_endstate_clusters : R.endstate_clusters) || []
raylu commented 2 years ago

ah, I didn't realize you built a patched lizzie. thanks!

lizgoban's image import is really nice and I think I got used to trial boards instead of game trees. thanks for the code snippet! I'll try to decide if I use lizgoban despite your warning or just use your patched lizzie

kaorahi commented 2 years ago

:-)

(Though implementation of this issue is easy (e1db84e3e), I'd like to defer it until it is really needed. Apps tend to be buggy if they have too many options that are not used by the developers themselves actually.)