Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pot not calculated correctly when the small blind also posts dead blind #25

Closed
DrkYoda opened this issue Oct 18, 2022 · 0 comments · Fixed by #28 or #27
Closed

Pot not calculated correctly when the small blind also posts dead blind #25

DrkYoda opened this issue Oct 18, 2022 · 0 comments · Fixed by #28 or #27
Assignees
@DrkYoda
Labels
bug Something isn't working

Comments

@DrkYoda
Copy link
Owner

DrkYoda commented Oct 18, 2022 •
edited
Loading

Description

Is an error generated from this bug?
Yes
Invalid pot size (0.15 vs pot 0.20 rake 0.00) for hand #xxxxxxxxx

To Reproduce

Steps to reproduce the behavior:

  1. Convert a Poker Now csv file to ohh that has a hand where the small blind posts a dead blind
  2. Import the ohh file into HM3

Expected Behavior

The pot size given by poker now should be equal to the pot size that HM3 calculates

Screenshots

image

Desktop

  • OS: Windows 10
    • Version: 10.0.19043
  • Language: Python, OHH, Holdem Manager
    • Version: 3.2.10, 1.2.2, 3.1.25
  • File: main.py
    • Version: 1.1.1

Additional Context

None
@DrkYoda DrkYoda added the bug Something isn't working label Oct 18, 2022
@DrkYoda DrkYoda self-assigned this Oct 18, 2022
@DrkYoda DrkYoda changed the title Pot not calculated correctly when the small blind is also posts dead blinds Pot not calculated correctly when the small blind also posts dead blind Oct 19, 2022
DrkYoda added a commit that referenced this issue Oct 19, 2022
@DrkYoda
Merge pull request #27 from DrkYoda:DrkYoda/issue25
507e570 
Fixes #25
This was referenced Oct 19, 2022
Merged
Open
@DrkYoda DrkYoda linked a pull request Nov 2, 2022 that will close this issue
Fixes #25 #27
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DrkYoda DrkYoda

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@DrkYoda