Closed barry-jin closed 3 years ago
FYI @icemelon9 @yzhliu , we added the monkey patch to try to fix the TVM conversion from the GluonNLP side.
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1502/7fe0c30d4552d753179f0bd4abecbcbb7a7987ed/index.html
Merging #1502 (7fe0c30) into master (09f3435) will decrease coverage by
0.16%
. The diff coverage is92.59%
.
@@ Coverage Diff @@
## master #1502 +/- ##
==========================================
- Coverage 86.54% 86.37% -0.17%
==========================================
Files 53 54 +1
Lines 7326 7349 +23
==========================================
+ Hits 6340 6348 +8
- Misses 986 1001 +15
Impacted Files | Coverage Δ | |
---|---|---|
src/gluonnlp/utils/tvm_utils.py | 92.30% <92.30%> (ø) |
|
src/gluonnlp/utils/misc.py | 54.64% <100.00%> (+0.22%) |
:arrow_up: |
src/gluonnlp/data/loading.py | 78.11% <0.00%> (-5.29%) |
:arrow_down: |
src/gluonnlp/data/tokenizers/huggingface.py | 71.54% <0.00%> (-0.29%) |
:arrow_down: |
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 09f3435...7fe0c30. Read the comment docs.
Description
Fix issue #1425. Update docker and enable tvm test.
Checklist
Essentials
Changes
Comments
cc @dmlc/gluon-nlp-team