[Mahjong] Implement Menzen

[OkanoShinri]

門前判定

これによって影響を受けるルール：

• riichi
• ron
• tsumo

[OkanoShinri]

#1048 でテスト資材追加

[OkanoShinri]

@sotetsuk
ci落ちているのがよくわからないのですが……時間かかりすぎってことでしょうか
手元のmake testは6分くらいで通ります

[sotetsuk]

んーとりあえず少し時間おいて再実行して様子みるとか？

[OkanoShinri]

んーとりあえず少し時間おいて再実行して様子みるとか？

雀のrandom playでコード143出して落ちているんですけど、そのテストが問題ではなく、マシン使い果たしている疑惑があります
actions/runner-images#6680

[OkanoShinri]

@sotetsuk
とりあえず一旦このPRどうにかしたいんですけど、どうしましょう……
あとは #1050 をやって基本ルールはカバーできると思っています

[sotetsuk]

再実行しました？👀@OkanoShinri

[sotetsuk]

どうしようというのはCIてすよね？？

[OkanoShinri]

d358162 を再実行したのですが同じ結果でした

CIのことです！

[sotetsuk]

d358162 を再実行したのですが同じ結果でした

CIのことです！

ちょっと今日の夜に確認するのでお待ちを…🙏

[sotetsuk]

んーでも僕の手元だとテスト落ちるな...

tests/test_mahjong.py::test_riichi Fatal Python error: Segmentation fault

[sotetsuk]

@OkanoShinri 一回、CIのコマンドを変更して、mahjong関連だけCI回るようにしてもらえますか？

https://github.com/sotetsuk/pgx/blob/main/.github/workflows/ci.yml#L29

[sotetsuk]

僕の手元だとmainでも test_mahjong.py がセグフォで死んでしまう......

[sotetsuk]

セグフォは別でissue立てました #1051

[OkanoShinri]

@OkanoShinri 一回、CIのコマンドを変更して、mahjong関連だけCI回るようにしてもらえますか？

https://github.com/sotetsuk/pgx/blob/main/.github/workflows/ci.yml#L29

了解です

[codecov]

Codecov Report

Merging #1045 (ab85f0a) into main (d9c99c8) will decrease coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1045      +/-   ##
==========================================
- Coverage   87.58%   87.30%   -0.29%     
==========================================
  Files          54       54              
  Lines        6035     6035              
==========================================
- Hits         5286     5269      -17     
- Misses        749      766      +17     

Files	Coverage Δ
pgx/_mahjong/_mahjong2.py	94.05% <100.00%> (ø)

... and 1 file with indirect coverage changes

[OkanoShinri]

mahjongのみだと通ってますね…

[sotetsuk]

@OkanoShinri いくつかテストを別PRで一旦コメントアウトしてみます

[sotetsuk]

@OkanoShinri よければマージしちゃってくださいー