search

mjx-project / mjx_mjai_translater

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

mjaiのactionをmjxのactionに変換する。 #40

Closed nissymori closed 3 years ago

nissymori commented 3 years ago

@sotetsuk

ということでaction_typeごとに照合の方法を変えて実装する方がいいと思ったのですがどうでしょうか。

if mjai_action["type"]=="dahai"
   for i in possible_actions
       ...
if mjai_action["type"]=="chi"
   for i in possible_actions
      ...
sotetsuk commented 3 years ago

それでいいと思います！possible actionにもツモ切りのタイプを加える予定なので、そこはTODOにしといて下さい🙇‍♂️

nissymori commented 3 years ago

テスト可能なものに関しては全てテストを通しました。

remain to be done

sotetsuk commented 3 years ago

No がないというのが少し気になりますね。ゲームエンジンからチーするか、しないか聞かれたとき、チーしない場合はmjaiではどう返すんですか？

nissymori commented 3 years ago

確かにそうですね。

良く考えたらnoneがnoにあたります。

その変換も実装します！

nissymori commented 3 years ago

NOの実装終わりました。

レビューお願い致します。

@sotetsuk

sotetsuk commented 3 years ago

いい感じだと思います！！ 🙆‍♀️ マージしちゃってください！！ @nissymori