mjx

mjx copied to clipboard

mjx::internal::KanAdded::TilesFromHand が壊れている

1

`mjx::internal::KanAdded::TilesFromHand` ``` std::vector KanAdded::TilesFromHand(std::uint16_t bits) { auto v = std::vector(); auto stolen = StolenTile(bits); for (int i = 0; i < 4; ++i) { auto t = At(bits, i); if...

habara-k

sotetsuk

sotetsuk

- 場風だけではわかりくい - whoも名前があまりよくないので変えたほうがいいかもしれない

sotetsuk

1000試合やっても990試合しか保存されなかった。並列でやったときに最後の一試合が保存されていない？

sotetsuk

- カカンのとき、どういう定義になる？ - 現状は3枚追加されている - 一方で、この3枚はclosed_tilesからopen時に移動するわけではない - open時にopen_tilesから移動するという前提で使われている

sotetsuk

python側でもかなり使うことが多いので、internalから上に上げる

sotetsuk

並び替えて先頭しかみていないので、優先順位が高く設定されている暗槓しか起きない ```cpp // 暗槓、加槓 if (mjx::internal::Any(selected.type(), {mjxproto::ACTION_TYPE_CLOSED_KAN, mjxproto::ACTION_TYPE_ADDED_KAN})) { auto prob = binary_model_prob( curr_hand, mjx::internal::Open(selected.open()), kan_model_); if (prob > 0.5) return selected; } ```

sotetsuk

現状、wheelでビルド後のテストがコケるので、C++に直書きしているが、`setup.py` の `package_data` に追記すればいけそう。 あるいは `constexpr` にしてこのままの方が便利か？

sotetsuk

今の所 `mjx::internal::State` を使っていて、 `mjx::State` がお飾りになっているので、主要な機能を移す

sotetsuk