Closed barry-jin closed 3 years ago
Merging #1428 into master will increase coverage by
0.20%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #1428 +/- ##
==========================================
+ Coverage 85.13% 85.33% +0.20%
==========================================
Files 53 53
Lines 6928 6928
==========================================
+ Hits 5898 5912 +14
+ Misses 1030 1016 -14
Impacted Files | Coverage Δ | |
---|---|---|
src/gluonnlp/data/loading.py | 83.39% <0.00%> (+5.28%) |
: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 f3d209f...679b57c. Read the comment docs.
The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1428/doc-warning/index.html
Description
Currently, GluonNLP CI for docs building will always fail because of the following errors.
WARNING: the mxtheme extension does not declare if it is safe for parallel reading, assuming it isn't - please ask the extension author to check and make it explicit
WARNING: doing serial read
We may have two possible solutions.mxtheme
extension to make parallel read safe.This PR will disable parallel building to remove these warnings.
Checklist
Essentials
Changes
Comments
cc @dmlc/gluon-nlp-team