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.

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

update from dmlc master #2

Merged
merged 2 commits into from
Jun 5, 2019
Merged

update from dmlc master #2

merged 2 commits into from
Jun 5, 2019

Conversation

xcgoner
Copy link
Owner

@xcgoner xcgoner commented Jun 5, 2019

Description

(Brief description on what this PR is about)

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

* move model definition to model/

* update tutorial

* add squad test

* add test argument

* refactor data folder

* fix tutorial

* fix lint

* fix import

* fix import

* update index.rst

* fix tutorial
…cab (#743)

* enable fp16 for ln. enable gelu

* manage processes

* manual killl

* support sentencepiece

* dont register signal handler

* add unigram sampling

* support comma for npz format

* fix file race condition

* switch to thread prefetcher

* fix pool.apply

* revert gelu support

* update documentation

* code cleanup

* avoid file download conflcit

* update doc

* fix bug

* bug fix

* more multi-processing

* fix lint

* update doc and fix lint

* fix test argument

* remove -test_bert_sentencepiece_sentences_transform()

* bug fix

* fix lint

* fix doc build
@xcgoner xcgoner merged commit fee3aac into xcgoner:master Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants