MahjongRepository / mahjong

Implementation of riichi mahjong related stuff (hand cost, shanten, agari end, etc.)
MIT License
377 stars 38 forks source link

Sanshoku Douko fails if one set is a kan #39

Closed joeygannon closed 3 years ago

joeygannon commented 3 years ago

This check fails if one of the sets is a kan, because the lists are different lengths. It may be preferable to just compare the first element of each set. https://github.com/MahjongRepository/mahjong/blob/416b5d914ca69354d3793eb800bc1944153ff9f8/mahjong/hand_calculating/yaku_list/sanshoku_douko.py#L47

Nihisil commented 3 years ago

Thanks for the bug report.