Closed sxjscience closed 3 years ago
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1435/try_reenable_test/index.html
Merging #1435 (3a54f78) into master (552d7e8) will increase coverage by
0.14%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #1435 +/- ##
==========================================
+ Coverage 85.31% 85.45% +0.14%
==========================================
Files 53 53
Lines 6987 6987
==========================================
+ Hits 5961 5971 +10
+ Misses 1026 1016 -10
Impacted Files | Coverage Δ | |
---|---|---|
src/gluonnlp/data/loading.py | 81.13% <0.00%> (-2.27%) |
:arrow_down: |
src/gluonnlp/data/tokenizers/subword_nmt.py | 78.50% <0.00%> (-0.94%) |
:arrow_down: |
src/gluonnlp/data/tokenizers/yttm.py | 81.89% <0.00%> (-0.87%) |
:arrow_down: |
src/gluonnlp/utils/parameter.py | 94.23% <0.00%> (+1.92%) |
:arrow_up: |
src/gluonnlp/attention_cell.py | 80.39% <0.00%> (+6.66%) |
: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 552d7e8...3a54f78. Read the comment docs.
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1435/try_reenable_test/index.html
Description
Try to fix https://github.com/dmlc/gluon-nlp/issues/1434
Checklist
Essentials
Changes
Comments
cc @dmlc/gluon-nlp-team