search

AlignmentResearch / KataGoVisualizer

MIT License
3 stars 1 forks source link

Human Attack and Analysis #52

Closed kellinpelrine closed 1 year ago

kellinpelrine commented 1 year ago

Adds human vs. bot games

Also includes code from @tomtseng to fix parsing in case of resignation, input manually from https://github.com/HumanCompatibleAI/KataGoVisualizer/commit/dbd9c0eb1ef55c4f90c46252aececed5bd20ee6f

netlify[bot] commented 1 year ago

Deploy Preview for goattack ready!

Name Link
Latest commit d5a35979fbcc1b2a39e0669cf3fd4432fa56781d
Latest deploy log https://app.netlify.com/sites/goattack/deploys/63d23f2d5da04b000d06aa36
Deploy Preview https://deploy-preview-52--goattack.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

AdamGleave commented 1 year ago

We should probably merge #49 first, rebase and re-run this (to pick up 2048->4096 visit change etc), then merge this PR. Either sequence is going to leave some conflicts, but I think #49 seems like the more fundamental one.

tomtseng commented 1 year ago

We should probably merge #49 first, rebase and re-run this (to pick up 2048->4096 visit change etc), then merge this PR. Either sequence is going to leave some conflicts, but I think #49 seems like the more fundamental one.

I merged #49 just now and fixed the merge conflicts in this PR

kellinpelrine commented 1 year ago

Sounds good on merging #49 / fixing conflicts, thank you Tom. I've added the position analysis games (https://deploy-preview-52--goattack.netlify.app/game-analysis#position-analysis) to make those available. Other than that should be good to go, will merge tomorrow morning if no further comments, or someone else feel free to.