fix tqdm usage

[CoinCheung]

Maybe this is the behavior that the authors would like to see, I guess.

[facebook-github-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

[facebook-github-bot]

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

[fmassa]

Is there a difference?

[CoinCheung]

I think so. On my platform of archlinux with python3.7, the following two pieces of code behaves differently (one has process bar, while the other does not).

from tqdm import tqdm
s = 1
for i, n in enumerate(tqdm(range(100000))):
    s += i

from tqdm import tqdm
s = 1
for i, n in tqdm(enumerate(range(100000))):
    s += i

[fmassa]

Oh, I see the difference now, thanks.

While this change would be ok, the fact that we use tqdm here is purely because I didn't bother changing it to a more logging-friendly approach back then.

While having the progress bar would be nice, it would also clutter much more some logs.
Anyway, I think we can merge this and remove the usage of tqdm later on.

[fmassa]

Thanks!