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.

Sign up for GitHub

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

Jump to bottom

tqdm dependency #24

Open

Djoels opened this issue Feb 22, 2018 · 2 comments

Djoels commented Feb 22, 2018

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 Jun 14, 2018

I agree it should go in the requirements.txt

4pal commented Sep 6, 2018 •

edited

Loading

requirements.txt
this satisfies the whole requirement issue

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