-
Notifications
You must be signed in to change notification settings - Fork 11
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
MCTSの可視化 #91
Comments
どちらも非常に興味深い機能だと思います。 |
ありがとうございます. 理解・改造しやすいことが TamaGo の魅力だと思いますから, 機能を欲ばってコードを肥大化させないようにと気にかけています. アニメーションのほうがやっつけ度が高いので後回しにして, ツリー表示について, まずは実装の方針をご相談願えないでしょうか. 以下の方針に何かコメントがあればお聞かせください.
また, ツリー表示を作っていたら下記がほしくなりました. こちらは別 PR にします. 本体の PR が形になるまで採否保留でも結構です.
|
実装方針確認しました。ツリー表示についてはその方針で良いと思います。 指定した回数探索するオプションについては、探索打ち切り機能を入れたタイミングで--visitsオプションの機能が微妙に変わってしまっているので確かにそのようなオプションは必要ですね。 tamago-read_sgfコマンドについても特別探索部やNN部に影響はありませんし、純粋に機能が拡張されるだけなのでPRが来たらマージする方向で考えています。 あと1点お手数をお掛けしますが、下記機能それぞれでissueを分けて作成していただけないでしょうか。
複数機能に絡むディスカッションは本issueで行いますが、個別機能の議論は各issueで行いたいという狙いがあります。 |
issueの作成、確認しました。 |
TamaGo を少し改造して MCTS の可視化を試しています. 現状のやっつけ実装からいくらか体裁を整えたら, マージの可能性はございますでしょうか?
教育やデモには良さそうに思われますが, プロジェクトの趣旨に合わなければご遠慮なく却下ください. 概念図ではない「現物」は見たことがなかったので, 個人的にはおもしろいです.
アニメーション
https://github.com/kaorahi/TamaGo/tree/mcts_step
mcts.mp4
(kaorahi/lizgoban#100 を参考にしました)
ツリー表示
https://github.com/kaorahi/TamaGo/tree/treeviz
100visits
1000 visits
10000 visits (principal variation 周辺のみ)
The text was updated successfully, but these errors were encountered: