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

Bert train fix #16

Open

hammad001 wants to merge 3 commits into rowanz:master from hammad001:bert-train-fix

hammad001 commented Mar 13, 2019

Include tf placeholders rather than constants for calculating bert embeddings. Otherwise, it violates the tensorflow's 2GB protocol.
fixes #15

hammad001 added 3 commits

March 4, 2019 17:47


Included bert embeddings and json in gitignore

edafee0


1. Changed saved model paths in eval file. \n 2. Included condition n…

cc187ab

…ot to convert metadata to gpu.


Changed tf constants to tf placeholders. Added session hooks.

d9480fa

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