Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

[BUGFIX] Update hash of wiki_cn_cased and wiki_multilingual_cased vocab #655

Merged
merged 2 commits into from Apr 11, 2019

Conversation

vanewu
Copy link
Contributor

@vanewu vanewu commented Apr 7, 2019

Description

Fix the bug mentioned in #652 . wiki_cn_cased vocab and wiki_multilingual_cased vocab have been converted. Now update the hash code of the relevant vocab.

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

Comments

@vanewu vanewu requested a review from szha as a code owner April 7, 2019 05:33
@mli
Copy link
Member

mli commented Apr 7, 2019

Job PR-655/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-655/1/index.html

@codecov
Copy link

codecov bot commented Apr 7, 2019

Codecov Report

Merging #655 into master will increase coverage by 0.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #655      +/-   ##
==========================================
+ Coverage   63.04%   63.73%   +0.68%     
==========================================
  Files         151      141      -10     
  Lines       13779    12902     -877     
==========================================
- Hits         8687     8223     -464     
+ Misses       5092     4679     -413
Flag Coverage Δ
#PR612 ?
#PR655 63.73% <ø> (?)
#master ?
#notserial 42.61% <ø> (+2.71%) ⬆️
#py2 63.49% <ø> (-0.37%) ⬇️
#py3 63.61% <ø> (+0.68%) ⬆️
#serial 49.19% <ø> (-0.22%) ⬇️

@codecov
Copy link

codecov bot commented Apr 7, 2019

Codecov Report

Merging #655 into master will increase coverage by 0.73%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #655      +/-   ##
==========================================
+ Coverage   63.04%   63.78%   +0.73%     
==========================================
  Files         151      141      -10     
  Lines       13779    12902     -877     
==========================================
- Hits         8687     8229     -458     
+ Misses       5092     4673     -419
Flag Coverage Δ
#PR612 ?
#PR655 63.78% <ø> (?)
#master 63.73% <ø> (ø) ⬆️
#notserial 42.61% <ø> (+2.71%) ⬆️
#py2 63.53% <ø> (-0.33%) ⬇️
#py3 63.62% <ø> (+0.68%) ⬆️
#serial 49.26% <ø> (-0.16%) ⬇️

Copy link
Member

@eric-haibin-lin eric-haibin-lin left a comment

Choose a reason for hiding this comment

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

Are there unit tests to validate this?

@vanewu
Copy link
Contributor Author

vanewu commented Apr 9, 2019

Are there unit tests to validate this?

Ok, I will add unit tests later.

@mli
Copy link
Member

mli commented Apr 10, 2019

Job PR-655/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-655/2/index.html

Copy link
Member

@eric-haibin-lin eric-haibin-lin left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@eric-haibin-lin eric-haibin-lin merged commit 2c8c376 into dmlc:master Apr 11, 2019
gigasquid pushed a commit to gigasquid/gluon-nlp that referenced this pull request May 6, 2019
…ab (dmlc#655)

* update hash of wiki_cn_cased and wiki_multilingual_cased

* add unit test for bertvocab
paperplanet pushed a commit to paperplanet/gluon-nlp that referenced this pull request Jun 9, 2019
…ab (dmlc#655)

* update hash of wiki_cn_cased and wiki_multilingual_cased

* add unit test for bertvocab
@vanewu vanewu deleted the BERTVocab branch June 12, 2019 09:57
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