search

Kyubyong / transformer

A TensorFlow Implementation of the Transformer: Attention Is All You Need
Apache License 2.0
4.24k stars 1.29k forks source link

tqdm dependency #24

Open Djoels opened 6 years ago

Djoels commented 6 years ago

Not really a big issue: I believe the tqdm module also belongs in the requirements.txt file?

PS: I really like your implementation! It was quite "painless" to get this working, compared to many other seq2seq repo's out there

👍 🥇

ruohoruotsi commented 6 years ago

I agree it should go in the requirements.txt

4pal commented 6 years ago

requirements.txt this satisfies the whole requirement issue