Closed barry-jin closed 3 years ago
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1444/update_tokenizer/index.html
I think the major tests have passed and we may wait https://github.com/dmlc/gluon-nlp/pull/1441
Merging #1444 (7bc31c4) into master (bda661b) will decrease coverage by
0.02%
. The diff coverage is50.00%
.
@@ Coverage Diff @@
## master #1444 +/- ##
==========================================
- Coverage 85.55% 85.53% -0.03%
==========================================
Files 53 53
Lines 6987 6987
==========================================
- Hits 5978 5976 -2
- Misses 1009 1011 +2
Impacted Files | Coverage Δ | |
---|---|---|
setup.py | 0.00% <ø> (ø) |
|
src/gluonnlp/data/tokenizers/huggingface.py | 72.06% <50.00%> (-0.84%) |
:arrow_down: |
src/gluonnlp/data/tokenizers/subword_nmt.py | 79.43% <0.00%> (+0.93%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update bda661b...7bc31c4. Read the comment docs.
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1444/update_tokenizer/index.html
Description
Try to solve #1431
Checklist
Essentials
Changes
Comments
cc @dmlc/gluon-nlp-team