Fix gross sum add tests#81
Open
paairs wants to merge 11 commits intoishikota:masterfrom
Open
Conversation
Fix: replace nose with pytest for Python 3.12 compatibility
- Fix A-2-3-4-5 wheel straight not recognized (issue ishikota#68/ishikota#69) - Fix flush comparison using only highest card instead of top 5 (issue ishikota#68) - Fix best 5-card combination not always chosen correctly from 7 (issue ishikota#78) - Fix duplicate cards in tests, ace rank 1 -> 14, update expected values - Add royal flush (treys class 0) to strength map
Fix: replace HandEvaluator with treys-based implementation
- Previously legal_actions() returned -1 for raise when max_raise < min_raise - Now returns all-in amount when max_raise > call amount (player can still raise) - Update test_legal_actions_when_short_of_money and test_need_amount_after_ante to reflect correct behaviour - Add test_raise_allin_when_stack_below_min_raise to cover the fixed case
Fix: allow all-in raise when stack is between call and min raise amount
PokerStars hand history generation
It should be gross amount, not net.
fix wrong calculated "Hero collected" -line.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.