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

PyPokerGUI not work with Python 3 #6

Open
hgenru opened this Issue Sep 1, 2017 · 2 comments

Comments

Projects
None yet
3 participants
@hgenru

hgenru commented Sep 1, 2017

First error is:

  File "round_state_html.generated.py", line 53, in _tt_execute
    for idx, player in zip(range(len(round_state['seats']))[:len(round_state['seats'])/2], round_state['seats']):  # round_state.html:22

But all code not prepared to work with Python 3

@Ns85

This comment has been minimized.

Show comment
Hide comment
@Ns85

Ns85 Sep 3, 2017

This fixed it for me:

first fix in round_state.html

<div id="seats-upper" class="row row-center">
{% for idx, player in zip(range(len(round_state['seats']))[:int(len(round_state['seats'])/2)], round_state['seats']) %}

second fix

<div id="seats-lower" class="row row-center">
{% for idx, player in zip(range(len(round_state['seats']))[int(len(round_state['seats'])/2):], round_state['seats'][int(len(round_state['seats'])/2):]) %}

Ns85 commented Sep 3, 2017

This fixed it for me:

first fix in round_state.html

<div id="seats-upper" class="row row-center">
{% for idx, player in zip(range(len(round_state['seats']))[:int(len(round_state['seats'])/2)], round_state['seats']) %}

second fix

<div id="seats-lower" class="row row-center">
{% for idx, player in zip(range(len(round_state['seats']))[int(len(round_state['seats'])/2):], round_state['seats'][int(len(round_state['seats'])/2):]) %}

chrisking added a commit to chrisking/PyPokerGUI that referenced this issue Sep 18, 2017

@chrisking

This comment has been minimized.

Show comment
Hide comment
@chrisking

chrisking Sep 18, 2017

#7 should be good to go. Thanks again @Ns85

chrisking commented Sep 18, 2017

#7 should be good to go. Thanks again @Ns85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment