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

Update create_training_instances to reduce for on documents #1231

Open

fernandoasevedo wants to merge 2 commits into google-research:master from fernandoasevedo:fernandoasevedo_update_create_training

+10 −5

fernandoasevedo commented Jun 2, 2021

A suggestion to remove a loop in all_documents at create_training_instances base on the following items:

remove len(all_documents) inside of for
Create a list current_document to collect non empty documents
Remove all_documents = [x for x in all_documents if x] because of item above

jacobdevlin-google and others added 2 commits

June 1, 2021 22:18


          Merge pull request google-research#1027 from iuliaturc-google/master

76a340a

Add links to 24 smaller BERT models.


          Merge branch 'master' of https://github.com/fernandoasevedo/bert

608896f

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