Skip to content
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

Common codebase to reproduce results #3

Merged
merged 17 commits into from
May 22, 2019
Merged

Common codebase to reproduce results #3

merged 17 commits into from
May 22, 2019

Conversation

zeynepakkalyoncu
Copy link
Contributor

@zeynepakkalyoncu zeynepakkalyoncu commented Apr 25, 2019 •
edited by Victor0118
Loading

  • Made minor edits to the robust04 branch to support training
  • Split functions into multiple files for readability
  • Added option to load local BERT model and tokenizer for servers without Internet access or memory restrictions
  • Updated instructions for running

Victor0118
Victor0118 reviewed Apr 25, 2019
View reviewed changes
util.py Outdated Show resolved Hide resolved
Victor0118
Victor0118 approved these changes Apr 25, 2019
View reviewed changes
Copy link
Owner

@Victor0118 Victor0118 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Victor0118
Copy link
Owner

LGTM

@Victor0118 Victor0118 merged commit 3543205 into Victor0118:robust04 May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Victor0118 Victor0118 Victor0118 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zeynepakkalyoncu @Victor0118