Yuki Kobayashi

Results 36 comments of Yuki Kobayashi

LizGobanのTamaGoサポート版が追加されたタイミングで追記。

どちらも非常に興味深い機能だと思います。 探索部や学習部の可読性が著しく低下したり、破壊的変更になるようなものでなければ基本的にPRは受け付けているので、本機能郡についてPRを出していただけるのであればマージして、次バージョンとしようと考えています。 コードについても細かい体裁やコメント等はこちらで理解しながら追記するので、気軽にPRを出していただければ幸いです。

実装方針確認しました。ツリー表示についてはその方針で良いと思います。 ツリーのダンプ機能についてはmcts/tree.pyに挿入して、ダンプからのグラフ表示機能は適当なディレクトリ(graphあたり?)を切って、その中に対応するスクリプトを入れるイメージですかね。 指定した回数探索するオプションについては、探索打ち切り機能を入れたタイミングで--visitsオプションの機能が微妙に変わってしまっているので確かにそのようなオプションは必要ですね。 もしかしたら既に実装済みかもしれませんが、まだコマンドラインオプションの追加対応に着手されていないのであれば、こちらで実装してdevelopブランチにpushしようと思います。 tamago-read_sgfコマンドについても特別探索部やNN部に影響はありませんし、純粋に機能が拡張されるだけなのでPRが来たらマージする方向で考えています。 置碁のサポートは多分そんなに難しくないですが、分岐のあるSGFを扱うとなると結構大変かもしれません。分岐のサポートの有無含め、どのように実装するかはタイミングを見て議論しましょう。 あと1点お手数をお掛けしますが、下記機能それぞれでissueを分けて作成していただけないでしょうか。 - ツリー表示 - アニメーション - tamago-read_sgf - 探索回数の厳密な指定オプション(--strict-visits) 複数機能に絡むディスカッションは本issueで行いますが、個別機能の議論は各issueで行いたいという狙いがあります。 またPRの際にどの機能かをマッピングできるようにしたいというものあります。

issueの作成、確認しました。 ありがとうございます。

requirements.txtのtorchのバージョンを合わせておいていただけますか。

ありがとうございます。問題点を理解できました。 PyTorchの修正が済んでからマージします。