Closed joeygannon closed 3 years ago
Thanks for the report. It was a tricky bug. The fix is available in the pre-release version: https://pypi.org/project/mahjong/1.2.0.dev6/
1.2.0.dev is different from the 1.1 version, there could be issues with migration.
I plan to finish the localization system in the next months and will release the 1.2.0 version with a migration guide together.
That's a tricky one indeed. Thank you for the quick fix!
This seems similar to #40, but I'm not sure if it's actually a duplicate. In this case it impacts the reported cost:
There can't be a closed_terminal_pon here because one of the 1m is in a chi and another is called for ron. Either open_terminal_pon (if you treat it as shanpon and divide it 111 234 44 555) or closed_pon (treat it as ryanmen and divide it 11 123 444 555) would be correct, and either reduces the fu to 30.