mjx-project / mjx_mjai_translater

Translation server between mjx and mjai protocols
MIT License
2 stars 0 forks source link

Mjai agentの手配の数がおかしい #126

Closed nissymori closed 3 years ago

nissymori commented 3 years ago

118

nissymori commented 3 years ago

actionが想定通りに送れていないことが判明。 mockしたテストではうまくいっているので差を考える。

スクリーンショット 2021-11-12 2 01 39

test caseにおいての結論

observationがmjaiのagentのactionと一致していないことによるbug observationをmjaiに合わせて作る?

nissymori commented 3 years ago

いかれケース①

スクリーンショット 2021-11-12 13 31 28 スクリーンショット 2021-11-12 13 31 53

target_id は2 一つ目のobservationで2までのツモが送られてきていて、 次の送られるactioinは0のツモから始まっている。 [2打牌, 3自摸, 3打牌]が抜けている。

考えられる原因

nissymori commented 3 years ago

いかれケース②

スクリーンショット 2021-11-12 13 50 23 スクリーンショット 2021-11-12 13 50 58

①と同様 原因は自分の打牌のactionが届いていなくてmjaiのagentの方の手配の数がおかしくなった。 observationは正しいので原因確定 →extract_differenceのバグ →修正

nissymori commented 3 years ago

上記のbugは解決しました。 20巡くらいは連続で動くようになりました。 まだいくつかbugの要素はあるみたいなので このPRは閉じて別のPRでbug fixにあたろうと思います。 問題なければmergeします。 @sotetsuk

sotetsuk commented 3 years ago

了解です!マージしちゃって下さいー🙏🙏🙏@nissymori

nissymori commented 3 years ago

レビューありがとうございます! 了解です! @sotetsuk