[Neural GPU] Running in Python 3.5

[arsenious]

Thank you for the code
I tried running the code in Python 3.5 (after converting the code using 2to3) , Tensorflow 1.01 but it ran into a couple of issues. Some fixes:

1. In wmt_utils.py, change line 44 to ensure everything inside re.compile is of type byte
   _WORD_SPLIT = re.compile( b"([" + _PUNCTUATION.encode() + b"])" )

2. In data_utils.py , change line 113 to keep k as an int
   k = (l-1)//2

I've just tested the bmul task and it seems to be running properly. I'll update as and when I find more issues.

[aselle]

It would be great if you could package this into a pull-request. Thanks!

[arsenious]

Hi aselle
To be honest, I'm relatively new to tensorflow as well as to git. I have used 2to3 to change all the scripts automatically. Wouldn't merging these changes to the repo cause issues for people using Python 2.7 ?

Thanks

[arsenious]

@aselle

Hi

I have made a pull request. Most of the changes were done by the python 2to3 script. However, changes to division operations performed on array indices had to be changed manually. ( / to // to ensure that the result was an integer). Please let me know what further actions I should take. I also signed the CLA.

Thanks

[aselle]

Awesome! Could you link to the pull request here? Or link the pull request to hear by Using "Fixes #1251" in the last commit of your git branch that you included in the pr. Thanks.

[arsenious]

Here is the link to the pull request. #1281

[karmel]

@lukaszkaiser Can you comment on this or close if no longer relevant?

[lukaszkaiser]

I think it's still a bug, just nobody corrected it (and the above PR didn't get merged).

[karmel]

@cclauss Note that this is an open issue on Python3 compatibility with a code owner available, if you want to contribute.

[jvishnuvardhan]

@lukaszkaiser Is this still an issue or was it solved by the PR? Thanks!

[lukaszkaiser]

From what I see the PR never got merged, so I don't think the problem is solved. But it may not be that relevant any more?

[tensorflowbutler]

Hi There,
We are checking to see if you still need help on this, as this seems to be considerably old issue. Please update this issue with the latest information, code snippet to reproduce your issue and error you are seeing.
If we don't hear from you in the next 7 days, this issue will be closed automatically. If you don't need help on this issue any more, please consider closing this.