Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

BERT QA notebook testing #191

Merged
merged 16 commits into from Jul 30, 2019
Merged

BERT QA notebook testing #191

merged 16 commits into from Jul 30, 2019

Conversation

catherine667
Copy link
Collaborator

@catherine667 catherine667 commented Jul 23, 2019

Description

BERT QA notebook testing
Fix broken links with SHA commit id in case the scripts will change

This test will run around 30mins (15mins training and 15mins tuning) with the current settings.

Related Issues

#25

Checklist:

  • My code follows the code style of this project, as detailed in our contribution guidelines.
  • I have added tests.
  • I have updated the documentation accordingly.

@review-notebook-app
Copy link

Check out this pull request on ReviewNB: https://app.reviewnb.com/microsoft/nlp/pull/191

You'll be able to see visual diffs and write comments on notebook cells. Powered by ReviewNB.

@miguelgfierro
Copy link
Member

This test will run around 30mins (15mins training and 15mins tuning) with the current settings.

Is it possible to change the settings so it takes 15min or so? or are we already hitting the lowest time possible?

@catherine667
Copy link
Collaborator Author

catherine667 commented Jul 25, 2019

This test will run around 30mins (15mins training and 15mins tuning) with the current settings.

Is it possible to change the settings so it takes 15min or so? or are we already hitting the lowest time possible?

I have tried different settings. Unfortunately, it seems like 30mins is the lowest time because the base model is large. I just let it fine tune on one epoch.

@saidbleik saidbleik merged commit b6083f4 into staging Jul 30, 2019
@catherine667 catherine667 deleted the liqun-bert-test branch July 30, 2019 17:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants