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.

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

Minraise calculation is wrong #13

Open
VinQbator opened this issue Feb 7, 2019 · 1 comment
Open

Minraise calculation is wrong #13

VinQbator opened this issue Feb 7, 2019 · 1 comment

Comments

@VinQbator
Copy link

When two players in heads-up minraising eachother, a sequence of total betsizes with 10/25 blinds: 10-25-60-110-etc come up. Should be 10-25-50-75-100-etc.

In env.py line 207

      elif move[0] == 'raise':
        self._player_bet(self._current_player, move[1] + self._current_player.currentbet)

Should be

      elif move[0] == 'raise':
        self._player_bet(self._current_player, move[1])

You are calculating the minraise as max(self._bigblind, self._lastraise + self._tocall) which yields us the total size of the bet with all previous bet sizes accounted for. No need to add our current bet.

@bdach
Copy link
Contributor

bdach commented Feb 7, 2019

Yeah, after some checking your version looks correct. Do you wish to submit a PR with this fix?

unaeat pushed a commit to unaeat/holdem that referenced this issue Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants