uda icon indicating copy to clipboard operation
uda copied to clipboard

Published 20 hours ago • verified publisher icon google-research

Python 2? Really?

Open impredicative opened this issue 5 years ago • 3 comments

The code is tested on Python 2.7 and Tensorflow 1.13.

Why use Python 2 for this and not Python 3? Python 2 is less than six months away from expiration. No one prefers to use Python 2 anymore.

impredicative avatar Jul 11 '19 02:07 impredicative

The text part of the code should be compatible with both python 2 and python 3, since BERT works for both. For the vision part, we used python 2 since AutoAugment was originally written in python 2.7. See this issue https://github.com/tensorflow/models/issues/6144

michaelpulsewidth avatar Jul 11 '19 18:07 michaelpulsewidth

There are some errors with py3, such as unicode and another differences between py2 and py3, have to find and fix them first.

Ritali avatar Dec 21 '19 09:12 Ritali

More specifically, trying text/scripts/prepro.sh with py3 results in TypeError: can't concat str to bytes from load_vocab. Are you guys considering updating to py3?

sshleifer avatar Jan 15 '20 19:01 sshleifer